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

Decouple from shared-workflows #2

Merged
merged 33 commits into from
May 1, 2024

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Apr 29, 2024

This PR makes the GHA workflows here independent of those in rapidsai/shared-workflows. It extracts the necessary logic into self-contained workflows so that we are not tied to the RAPIDS branching/versioning strategy. In addition, it splits the core logic into a shared workflow that can be used for both building and publishing, which will enable downstream work to actually start publishing packages.

@vyasr vyasr marked this pull request as ready for review April 30, 2024 20:02
@vyasr vyasr requested a review from a team as a code owner April 30, 2024 20:02
@vyasr vyasr changed the title Workflow updates Decouple from shared-workflows Apr 30, 2024
.github/workflows/build_and_test.yaml Outdated Show resolved Hide resolved
.github/workflows/build_and_test.yaml Outdated Show resolved Hide resolved
.github/workflows/build_and_test.yaml Outdated Show resolved Hide resolved
.github/workflows/build_and_test.yaml Outdated Show resolved Hide resolved
@vyasr vyasr merged commit d156258 into rapidsai:main May 1, 2024
12 checks passed
@vyasr vyasr deleted the feat/decouple_shared_workflows branch May 1, 2024 19:57
vyasr added a commit that referenced this pull request May 1, 2024
This PR updates the build job that occurs on pushes to main to also use
the reusable workflow from #2. It also adds logic to check if a push to
main has modified the VERSION file. If so, the new package is published
if tests pass.
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.

3 participants