Skip to content

Conversation

@Bahati308
Copy link
Contributor

@Bahati308 Bahati308 commented Jan 30, 2026

ci(formulus-android): merge formplayer assets into android workflow

Description

  • Merge the separate Formplayer assets workflow into the Formulus Android workflow as two dependent jobs.
  • Build Formplayer assets in a dedicated build-formplayer-assets job and pass them to the Android build via an artifact.
  • Remove the old update-formplayer-assets workflow and stop committing built assets to the repo.
  • Ignore formplayer_dist in .gitignore so built assets are no longer tracked in git.

Type of Change

  • Bug Fix
  • New Feature / Enhancement
  • Refactor / Code Cleanup
  • Documentation Update
  • Maintenance / Chore
  • Other (please specify): CI/CD workflow restructuring

Component(s) Affected

  • formulus (React Native mobile app)
  • formulus-formplayer (React web app)
  • synkronus (Go backend server)
  • synkronus-cli (Command-line utility)
  • Documentation
  • DevOps / CI/CD
  • Other:

Related Issue(s)

Closes/Fixes/Resolves: N/A


Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manually tested
  • Tested on multiple platforms (if applicable)
  • Not applicable

Manual verification: workflow YAML passes syntax/structure checks and job dependencies are correctly configured.


Breaking Changes

  • This PR introduces breaking changes
  • This PR does NOT introduce breaking changes

If breaking changes, please describe migration steps:

N/A


Documentation Updates

  • Documentation has been updated
  • Documentation update is not required

Checklist

  • Code follows project style guidelines
  • All existing tests pass
  • New tests added for new functionality
  • PR title follows Conventional Commits format

Thank you for contributing to Open Data Ensemble (ODE)!

najuna-brian
najuna-brian previously approved these changes Jan 30, 2026
Copy link
Contributor

@najuna-brian najuna-brian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Bahati308
This simplifies the CI/CD setup, removes auto-commit complexity, and uses artifacts appropriately.
The consolidation makes the workflow easier to maintain and understand.

@najuna-brian
Copy link
Contributor

Thanks for the docs update

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