Skip to content

fix: harden release workflow and resolve PR #3 conflicts - #4

Merged
cursor[bot] merged 5 commits into
mainfrom
cursor/review-release-workflow-2e61
Jul 11, 2026
Merged

fix: harden release workflow and resolve PR #3 conflicts#4
cursor[bot] merged 5 commits into
mainfrom
cursor/review-release-workflow-2e61

Conversation

@Alpaca233114514

@Alpaca233114514 Alpaca233114514 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Merged main into the release-workflow changes and resolved the README conflict with the trilingual docs.
  • Gated the Web CI/release job behind ENABLE_WEB_BUILD until unconditional dart:io imports are fixed.
  • Marked any hyphenated semver tag as a GitHub prerelease.
  • Use flutter analyze --no-fatal-infos so pre-existing info-level lints do not block CI.

Review notes

Same tip as updated #3. Prefer merging one of them after CI is green; close the other as duplicate.

Testing

  • Validated release.yml YAML syntax.
  • Waiting on GitHub Actions quality job after the analyze flag fix.
Open in Web Open in Cursor 

Alpaca233114514 and others added 5 commits July 11, 2026 12:47
- Add GitHub Actions workflow for multi-platform builds and releases (Windows, macOS, Linux, Android, iOS, Web)
- Add PowerShell setup script for Windows development environment initialization
- Add VS Code configuration files for Dart/Flutter development
- Rewrite README with comprehensive quick start guide, platform notes, and release procedures
- Update dependencies (meta 1.17.0 → 1.18.0, test_api 0.7.10 → 0.7.11)
- Remove sqlite3 system source configuration from pubspec.yaml to use precompiled binary
Improve GitHub Actions release workflow: include workflow name in concurrency group and only cancel in-progress runs for pull_request events; skip full platform builds (android/windows/linux/macos/ios) during pull_request runs. Add robust tag verification using gh CLI to ensure a remote tag exists and resolves to the exact commit targeted by the workflow (fail if mismatched) before uploading release assets. Other edits: remove hardcoded dart.flutterSdkPath from .vscode settings, clarify Flutter SDK path and CI-fixed version in README, and adjust setup.ps1 log text for Drift code generation. These changes prevent accidental release-asset overwrites and make local SDK config guidance clearer.
Merge main into the release-workflow branch, combine the trilingual README
with onboarding/CI docs, gate the web build behind ENABLE_WEB_BUILD until
unconditional dart:io imports are fixed, and treat any hyphenated semver
tag as a prerelease.

Co-authored-by: ALPACA LI <liyuanstudio273@gmail.com>
Existing prefer_interpolation / avoid_print / use_build_context_synchronously
infos were failing `flutter analyze` and blocking the new release workflow.

Co-authored-by: ALPACA LI <liyuanstudio273@gmail.com>
@cursor
cursor Bot merged commit 597c7d2 into main Jul 11, 2026
18 checks passed
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.

2 participants