Refactor project configuration and CI/CD workflows #981
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Remove firebaserc files * Remove firebase exports * Remove hard-coded configs in favour of env variables * Add support for dotenvx * Add dotenvx pre-commit to husky hook * Add env-configs submodule * Add dotenvx support to Vite * Fix env var boolean parsing * Remove markRaw firekit config * Rename check-format cmd to format:check * Simplify package dev and build cmds
roar-dashboard-e2e
|
||||||||||||||||||||||||||||
| Project |
roar-dashboard-e2e
|
| Branch Review |
feat/project-config
|
| Run status |
|
| Run duration | 02m 18s |
| Commit |
|
| Committer | Maximilian Oertel |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
0
|
Pending
|
0
|
Skipped
|
0
|
Passing
|
87
|
| View all changes introduced in this branch ↗︎ | |
* Fix dotenvx support for Cypress * Remove obsolete sharp dependency * Adjust vite preview port config * Adjust vitest config to support test env * Temporarily disable cypress workflows * Introduce build action * Introduce deploy action * Introduce setup-node-environment action * Introduce deploy-firestore-config action * Introduce update-deployment-manifest action * Introduce whitelist-appcheck-environment action * Refactor ci workflow to use internal actions * Introduce deploy-development workflow * Introduce deploy-staging workflow * Merge firebase.json * Delete function symlinks * Add production deployment workflow * Remove redundant conditionals * Remove obsolete workflows * Add Sentry release to production workflow * Add project matrix fallback for manifest update steps * Temporarily disable E2E tests * Add debug step * Bump env-configs * Fix app deployment action * Fix deployment URL output * Re-enable workflow steps * Bump env-configs submodule * Add missing dotenvx key
Update `feat/project-config` from `main`
Coverage Report
File CoverageNo changed files found. |
* Refactor ci workflow to use internal actions * Enable component tests * Remove redundant env var mapping * Add participant and partner auth specs * Move assent form test to legal spec * Introduce waitForParticipantHomepage cmd * Remove timeout from navigateTo cmd * Improve auth cmds * Update .env.test.example * Remove legacy clever sign-in helper * Combine score report specs * Remove obsolete logout spec * Combine progress report specs * Refactor list orgs specs * Refactor individual report specs * Remove left-over whitelisted route * Add missing data-cy attrs * Remove obsolete specs and adjust file names * Fix base url usage * Remove arbitrary timeout overrides * Remove @vite-pwa/assets-generator due to dependency conflicts * Add temporary debug steps * Fix debug cmd * Remove debug steps * Add missing cypress-wait-until dependency * Temporarily disable component tests * Temporarily change e2e matrix * Fix app route constant * Fix route definition * Fix participant homepage test ids * Fix test id override * Move assessment tests into dedicated directory * Standardize assessment specs * Refactor parent signup invitation code specs * Add parent test suite to E2E CI step * Enable container parallelisation for E2E tests * Re-enable browser matrix for E2E tests * Remove unused old-tests directory * Remove individual E2E test workflows * Simplify helper functions wait statements * Fix dotenvx support for Cypress * Remove obsolete sharp dependency * Adjust vite preview port config * Add debug step * Bump env-configs * Fix app deployment action * Re-enable workflow steps * Fix formatting offenses * Introduce cy.waitForAssessmentReadyState cmd * Fix missing env var from example file * Restore SWR test timeouts * Fix config files * Fix HomeParticipant rebase conflicts * Fix remaining merge conflicts and formatting offenses * Remove obsolete Cypress cmd * Fix navbar test ids * Fix selectors and test ids * Add missing data table test ids * Fix report tests * Bump Cypress docker image * Switch config error to console log * Re-enable component tests * Align auth test IDs * Fix component test path * Fix docker image name typo
richford
approved these changes
Jan 31, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This feature branch contains the following collection of chained PRs that have been independently reviewed:
feat/project-configfrommain#1013Ref https://github.com/yeatmanlab/roar-project-management/issues/377