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

Refactor CI using central flow #230

Merged
merged 3 commits into from
Jan 9, 2025
Merged

Refactor CI using central flow #230

merged 3 commits into from
Jan 9, 2025

Conversation

garryod
Copy link
Member

@garryod garryod commented Jan 7, 2025

Re-structures the CI workflow to use a central dispatch workflow - this will be useful when using release please as it allows state to be passed between steps

@garryod garryod added the github_actions Pull requests that update GitHub Actions code label Jan 7, 2025
@garryod garryod requested a review from GDYendell January 7, 2025 16:37
@garryod garryod self-assigned this Jan 7, 2025
@garryod garryod force-pushed the refactor-ci branch 7 times, most recently from 9759944 to 73408f0 Compare January 7, 2025 16:52
@garryod garryod marked this pull request as ready for review January 8, 2025 10:43
Copy link

@GDYendell GDYendell left a comment

Choose a reason for hiding this comment

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

A few comments. The flow of jobs makes sense and it's much neater this way.

.github/workflows/_bundler_docs.yaml Show resolved Hide resolved
.github/workflows/_bundler_lint.yaml Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
@garryod garryod force-pushed the refactor-ci branch 11 times, most recently from 5199720 to 395b8fe Compare January 8, 2025 16:56
@garryod garryod force-pushed the refactor-ci branch 3 times, most recently from 45617b2 to 97629b6 Compare January 8, 2025 17:07
GDYendell
GDYendell previously approved these changes Jan 9, 2025
@garryod garryod force-pushed the refactor-ci branch 5 times, most recently from 0150e92 to b3fb476 Compare January 9, 2025 10:28
@garryod garryod merged commit dadd6df into main Jan 9, 2025
33 checks passed
@garryod garryod deleted the refactor-ci branch January 9, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants