Skip to content

Prove WNS delivery on the Windows runner with real pushes - #14

Merged
LucaCappelletti94 merged 3 commits into
t1-new-intent-tapsfrom
w0-windows-ci
Oct 10, 2026
Merged

LucaCappelletti94 merged 3 commits into
t1-new-intent-tapsfrom
w0-windows-ci

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Owner

The Windows backend from #9 only had compile and missing-runtime checks (#12). This adds a CI job that installs the Windows App SDK 2.5 runtime on GitHub's Windows Server 2025 runner, runs a small unpackaged probe app built on pushups, gets a token from the repository's Entra ID app registration, and sends real raw pushes through WNS, following Microsoft's push quickstart.

On the branch it passed in run 37944507867. A push reached the running app as Event::Message on channels made with both the service principal's Object ID and the Application ID, and a push to the closed app made Windows start it and run its background handler with started_app set. The README now calls Windows available and says a Windows 11 desktop has not been checked yet.

The job needs the four WNS_* secrets, so it runs on pushes to main, on pull requests from this repository and by hand, and skips pull requests from forks.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: LucaCappelletti94/coderabbit/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: fe7f3a9b-ac32-4e0f-933c-9152f4f68e7f

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.91%. Comparing base (b1a32d9) to head (66bf083).

Additional details and impacted files
@@                 Coverage Diff                 @@
##           t1-new-intent-taps      #14   +/-   ##
===================================================
  Coverage               91.91%   91.91%           
===================================================
  Files                      33       33           
  Lines                    2844     2844           
  Branches                   38       38           
===================================================
  Hits                     2614     2614           
  Misses                    203      203           
  Partials                   27       27           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud

sonarqubecloud Bot commented Oct 9, 2026

Copy link
Copy Markdown

@LucaCappelletti94
LucaCappelletti94 merged commit 8c15151 into main Oct 10, 2026
44 of 46 checks passed
@LucaCappelletti94
LucaCappelletti94 deleted the w0-windows-ci branch October 10, 2026 12:11
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.

1 participant