Skip to content

ci(mobile): fix YAML cache syntax in mobile CI workflow - #126

Merged
Decuayer merged 2 commits into
mainfrom
fix/mobile-ci-workflow
Aug 28, 2026
Merged

ci(mobile): fix YAML cache syntax in mobile CI workflow#126
Decuayer merged 2 commits into
mainfrom
fix/mobile-ci-workflow

Conversation

@Decuayer

@Decuayer Decuayer commented Aug 28, 2026

Copy link
Copy Markdown
Owner

🎯 Jira / Task Reference

  • Jira Issue: SDDC-5 (EPIC 5: CI/CD & Deployment)
  • Component: [mobile | infra]

📝 Summary of Changes

Fixes the GitHub Actions Mobile CI workflow parsing error by simplifying the Flutter caching configuration.

  • Removed invalid YAML quote escape sequence (\') in mobile-ci.yml.
  • Standardized Flutter SDK caching using native subosito/flutter-action@v2 automated cache resolution.
  • Verified that flutter analyze and flutter test execute cleanly in the CI runner.

🏷️ Type of Change

  • 🐛 Bug Fix: Non-breaking fix for an identified defect
  • 🛠️ Chore / Infra: Dependencies, Docker, CI/CD, or tooling updates

🧪 Testing & Verification

Automated Tests

  • Mobile tests executed: flutter test (Passed: 100%)
  • Linter & formatting passed: flutter analyze (No issues found)

Manual Test Steps

  1. Push branch to remote.
  2. Verify that GitHub Actions pipeline triggers and compiles without YAML syntax errors.
  3. Confirm green build on Mobile CI.

📋 Pre-Merge Checklist

  • My code adheres to the project's Clean Architecture & style guidelines.
  • I have self-reviewed my own code before requesting review.
  • Database migrations are backwards-compatible (if applicable).
  • Sensitive secrets / tokens are NOT committed (verified .env exclusions).
  • Jira status updated to In Review.

@Decuayer
Decuayer merged commit 9d3867e into main Aug 28, 2026
1 check 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.

1 participant