-
Notifications
You must be signed in to change notification settings - Fork 264
Add Windows Support #532
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Windows Support #532
Changes from all commits
8e52a09
d4f0783
2706370
d561e46
948c474
8e2dd69
9d818ef
beb7637
d6e4c5d
5b15859
3dadfea
2e08a96
572e99a
2392c4b
3f5ee17
a1e40ab
d421866
8584e9a
a4dc731
d719a6c
f5b9a2f
4ff643d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -15,10 +15,8 @@ jobs: | |||||||||||||||||||||||||||||
| fail-fast: false | ||||||||||||||||||||||||||||||
| matrix: | ||||||||||||||||||||||||||||||
| include: | ||||||||||||||||||||||||||||||
| - platform: macos-latest | ||||||||||||||||||||||||||||||
| args: "--target aarch64-apple-darwin" | ||||||||||||||||||||||||||||||
| - platform: macos-latest | ||||||||||||||||||||||||||||||
| args: "--target x86_64-apple-darwin" | ||||||||||||||||||||||||||||||
| - platform: windows-latest | ||||||||||||||||||||||||||||||
| args: "--target x86_64-pc-windows-msvc" | ||||||||||||||||||||||||||||||
|
Comment on lines
17
to
+19
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Matrix now only includes Windows, removing macOS builds. The matrix has been replaced to only target Windows, effectively removing macOS builds from this workflow. This contradicts the PR's goal of "preserving macOS compatibility" and would prevent macOS releases from being published. The matrix should include both platforms: 🐛 Proposed fix to restore macOS targets matrix:
include:
- platform: windows-latest
args: "--target x86_64-pc-windows-msvc"
+ - platform: macos-latest
+ args: "--target aarch64-apple-darwin"
+ - platform: macos-latest
+ args: "--target x86_64-apple-darwin"📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||
| runs-on: ${{ matrix.platform }} | ||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||
| RELEASE_TAG: ${{ github.ref_name }} | ||||||||||||||||||||||||||||||
|
|
@@ -29,7 +27,7 @@ jobs: | |||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - uses: dtolnay/rust-toolchain@stable | ||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||
| targets: aarch64-apple-darwin,x86_64-apple-darwin | ||||||||||||||||||||||||||||||
| targets: aarch64-apple-darwin,x86_64-apple-darwin,x86_64-pc-windows-msvc | ||||||||||||||||||||||||||||||
|
Comment on lines
28
to
+30
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Apple Darwin targets installed but matrix has no macOS runner. The Rust toolchain installs 🛠️ Proposed fix: make targets platform-specific - uses: dtolnay/rust-toolchain@stable
with:
- targets: aarch64-apple-darwin,x86_64-apple-darwin,x86_64-pc-windows-msvc
+ targets: ${{ contains(matrix.platform, 'macos') && 'aarch64-apple-darwin,x86_64-apple-darwin' || 'x86_64-pc-windows-msvc' }}Alternatively, define 🧰 Tools🪛 zizmor (1.25.2)[error] 28-28: unpinned action reference (unpinned-uses): action is not pinned to a hash (required by blanket policy) (unpinned-uses) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||
| - uses: swatinem/rust-cache@v2 | ||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||
| workspaces: "./src-tauri -> target" | ||||||||||||||||||||||||||||||
|
|
@@ -41,6 +39,7 @@ jobs: | |||||||||||||||||||||||||||||
| - name: Bundle plugins | ||||||||||||||||||||||||||||||
| run: bun run bundle:plugins | ||||||||||||||||||||||||||||||
| - name: Verify bundled plugins | ||||||||||||||||||||||||||||||
| shell: bash | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
| COUNT=$(find src-tauri/resources/bundled_plugins -maxdepth 2 -name plugin.json | wc -l | tr -d ' ') | ||||||||||||||||||||||||||||||
| if [[ "$COUNT" -lt 1 ]]; then | ||||||||||||||||||||||||||||||
|
|
@@ -49,6 +48,7 @@ jobs: | |||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - name: Validate release tag | ||||||||||||||||||||||||||||||
| shell: bash | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
| if [[ -z "$RELEASE_TAG" ]]; then | ||||||||||||||||||||||||||||||
| echo "Missing RELEASE_TAG (push a v* tag)." | ||||||||||||||||||||||||||||||
|
|
@@ -60,6 +60,7 @@ jobs: | |||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - name: Validate app version matches tag | ||||||||||||||||||||||||||||||
| shell: bash | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
| TAG_VERSION="${RELEASE_TAG#v}" | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
|
|
@@ -81,11 +82,16 @@ jobs: | |||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - name: Import Apple Developer Certificate | ||||||||||||||||||||||||||||||
| if: matrix.platform == 'macos-latest' | ||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||
| APPLE_CERTIFICATE: ${{ secrets.APPLE_CERTIFICATE }} | ||||||||||||||||||||||||||||||
| APPLE_CERTIFICATE_PASSWORD: ${{ secrets.APPLE_CERTIFICATE_PASSWORD }} | ||||||||||||||||||||||||||||||
| KEYCHAIN_PASSWORD: ${{ secrets.KEYCHAIN_PASSWORD }} | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
| if [[ -z "$APPLE_CERTIFICATE" ]]; then | ||||||||||||||||||||||||||||||
| echo "Skipping Apple Certificate import because secret is empty." | ||||||||||||||||||||||||||||||
| exit 0 | ||||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||
| echo "$APPLE_CERTIFICATE" | base64 --decode > certificate.p12 | ||||||||||||||||||||||||||||||
| security create-keychain -p "$KEYCHAIN_PASSWORD" build.keychain | ||||||||||||||||||||||||||||||
| security default-keychain -s build.keychain | ||||||||||||||||||||||||||||||
|
|
@@ -95,6 +101,24 @@ jobs: | |||||||||||||||||||||||||||||
| security set-key-partition-list -S apple-tool:,apple:,codesign: -s -k "$KEYCHAIN_PASSWORD" build.keychain | ||||||||||||||||||||||||||||||
| rm certificate.p12 | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - name: Handle missing updater key (for forks) | ||||||||||||||||||||||||||||||
| shell: bash | ||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||
| TAURI_KEY: ${{ secrets.TAURI_SIGNING_PRIVATE_KEY }} | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
| if [[ -z "$TAURI_KEY" ]]; then | ||||||||||||||||||||||||||||||
| echo "HAS_TAURI_KEY=false" >> $GITHUB_ENV | ||||||||||||||||||||||||||||||
| node -e " | ||||||||||||||||||||||||||||||
| const fs = require('fs'); | ||||||||||||||||||||||||||||||
| const conf = JSON.parse(fs.readFileSync('./src-tauri/tauri.conf.json')); | ||||||||||||||||||||||||||||||
| if(conf.bundle) conf.bundle.createUpdaterArtifacts = false; | ||||||||||||||||||||||||||||||
| if(conf.plugins && conf.plugins.updater) delete conf.plugins.updater; | ||||||||||||||||||||||||||||||
| fs.writeFileSync('./src-tauri/tauri.conf.json', JSON.stringify(conf, null, 2)); | ||||||||||||||||||||||||||||||
| " | ||||||||||||||||||||||||||||||
| else | ||||||||||||||||||||||||||||||
| echo "HAS_TAURI_KEY=true" >> $GITHUB_ENV | ||||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - uses: tauri-apps/tauri-action@v0 | ||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||
|
|
@@ -114,10 +138,12 @@ jobs: | |||||||||||||||||||||||||||||
| releaseName: ${{ env.RELEASE_TAG }} | ||||||||||||||||||||||||||||||
| releaseDraft: false | ||||||||||||||||||||||||||||||
| prerelease: false | ||||||||||||||||||||||||||||||
| includeUpdaterJson: true | ||||||||||||||||||||||||||||||
| includeUpdaterJson: ${{ env.HAS_TAURI_KEY == 'true' }} | ||||||||||||||||||||||||||||||
| args: ${{ matrix.args }} | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| - name: Verify updater assets uploaded | ||||||||||||||||||||||||||||||
| if: env.HAS_TAURI_KEY == 'true' | ||||||||||||||||||||||||||||||
| shell: bash | ||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,5 +1,18 @@ | ||||||||||||||
| # Changelog | ||||||||||||||
|
|
||||||||||||||
| ## v0.6.37 | ||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Align release notes metadata with the shipped version and canonical repo links. Line 3 documents Also applies to: 12-14 🤖 Prompt for AI Agents |
||||||||||||||
|
|
||||||||||||||
| ### Bug Fixes | ||||||||||||||
| - fix: cursor db path on windows and os-agnostic error messages by @Native Muttaqien | ||||||||||||||
|
|
||||||||||||||
| --- | ||||||||||||||
|
|
||||||||||||||
| ### Changelog | ||||||||||||||
|
|
||||||||||||||
| **Full Changelog**: [v0.6.36...v0.6.37](https://github.com/samlehoy/openusage/compare/v0.6.36...v0.6.37) | ||||||||||||||
|
|
||||||||||||||
| - [3f5ee17](https://github.com/samlehoy/openusage/commit/3f5ee17) fix: cursor db path on windows and os-agnostic error messages by @Native Muttaqien | ||||||||||||||
|
Comment on lines
+12
to
+14
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: Incorrect repository owner in v0.6.37 changelog links breaks URLs Prompt for AI agents
Suggested change
|
||||||||||||||
|
|
||||||||||||||
| ## v0.6.24 | ||||||||||||||
|
|
||||||||||||||
| ### New Features | ||||||||||||||
|
|
||||||||||||||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P1: Release workflow matrix removed all macOS build targets, breaking macOS releases while the Apple certificate step still expects a
macos-latestplatform that no longer exists.Prompt for AI agents