Skip to content
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

Separate webpage build from data processing steps in action #262 #272

Merged
merged 3 commits into from
Mar 11, 2025

Conversation

richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Mar 9, 2025

Do not Merge
This needs to be Tested first

issue #262

The workflow is split into two: data processing (runs daily/on-demand) generates necessary files (contributors, resources, analytics) and saves them as artifacts. Deployment (triggers on PRs/commits/daily) uses these pre-built artifacts to quickly build and deploy the site, avoiding redundant data steps.
This ensures faster PR checks and prevents data-related failures from blocking site deployment.

@richarddushime richarddushime requested a review from a team as a code owner March 9, 2025 16:48
@github-actions github-actions bot added the cicd Relevant to GitHub workflows label Mar 9, 2025
Copy link

github-actions bot commented Mar 9, 2025

👍 All image files/references (if any) are in webp format, in line with our policy.

@LukasWallrich
Copy link
Contributor

Looks very good to me - let me know when you think it's ready to merge.

@richarddushime
Copy link
Contributor Author

Looks very good to me - let me know when you think it's ready to merge.

Great , I think we can merge this prior to the time and see How All CI s will behave

Hoping to join the tomorrow call

Copy link
Contributor

@LukasWallrich LukasWallrich left a comment

Choose a reason for hiding this comment

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

Looks good throughout - I believe full test will only happen after merge.

@LukasWallrich LukasWallrich merged commit 695b492 into master Mar 11, 2025
4 checks passed
@LukasWallrich LukasWallrich deleted the separate-build branch March 11, 2025 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cicd Relevant to GitHub workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants