Skip to content

chore: add ci formatting checks and csharpier cleanup#160

Merged
fadi-george merged 5 commits into
mainfrom
fg/ci
Mar 9, 2026
Merged

chore: add ci formatting checks and csharpier cleanup#160
fadi-george merged 5 commits into
mainfrom
fg/ci

Conversation

@fadi-george
Copy link
Copy Markdown
Contributor

Description

One Line Summary

Add a pull request formatting check, apply CSharpier cleanup across the repo, remove old unused tests, and add local PR conventions.

Details

Motivation

Bring basic formatting validation into CI and normalize formatting across the SDK so future changes are easier to review and maintain.

Scope

This updates repository tooling and formatting across the SDK, bindings, and demo app. It also removes an old unused test file and adds a local Cursor PR convention rule. No behavior changes are intended.

Testing

Unit testing

No new unit tests were added. This branch removes an old unused test file.

Manual testing

Not run.

Affected code checklist

  • Notifications
  • Display
  • Open
  • Push Processing
  • Confirm Deliveries
  • Outcomes
  • Sessions
  • In-App Messaging
  • REST API requests
  • Public API changes

Checklist

Overview

  • I have filled out all REQUIRED sections above
  • PR does one thing
  • Any Public API changes are explained in the PR details and conform to existing APIs

Testing

  • I have included test coverage for these changes, or explained why they are not needed
  • All automated tests pass, or I explained why that is not possible
  • I have personally tested this on my device, or explained why that is not possible

Final pass

  • Code is as readable as possible.
  • I have reviewed this PR myself, ensuring it meets each checklist item

Made with Cursor

Comment thread .github/workflows/ci.yml Fixed
…in permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@fadi-george fadi-george merged commit 2710e85 into main Mar 9, 2026
7 checks passed
@fadi-george fadi-george deleted the fg/ci branch March 9, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants