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

[CI] Adding CI Workflows: checks, changed-files, builds #6

Merged
merged 92 commits into from
Nov 1, 2024

Conversation

nv-rliu
Copy link
Contributor

@nv-rliu nv-rliu commented Oct 11, 2024

Closes https://github.com/rapidsai/graph_dl/issues/636
Closes https://github.com/rapidsai/graph_dl/issues/637

This PR adds the following jobs in CI:

  • changed-files
  • conda-python-build
  • wheel-build-nx-cugraph

@nv-rliu nv-rliu added breaking Introduces a breaking change improvement Improves an existing functionality ci labels Oct 11, 2024
@nv-rliu nv-rliu mentioned this pull request Oct 14, 2024
@nv-rliu nv-rliu changed the title [Don't Merge] PR to Test GHA Workflows [Don't Merge] PR to Test build_python.sh Oct 14, 2024
@nv-rliu nv-rliu changed the title [Don't Merge] PR to Test build_python.sh [Don't Merge] PR to Test checks and build_python.sh Oct 14, 2024
@nv-rliu
Copy link
Contributor Author

nv-rliu commented Oct 31, 2024

@KyleFromNVIDIA requesting another review!

Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

@KyleFromNVIDIA I've joined here as well because @nv-rliu asked me to (he and I have been pairing on this repo). Feel free to dismiss your review and take yourself off the reviewers list if you'd like to focus on other things.

@nv-rliu I left some comments for your consideration.

.github/workflows/pr.yaml Outdated Show resolved Hide resolved
.github/workflows/pr.yaml Outdated Show resolved Hide resolved
ci/build_python.sh Outdated Show resolved Hide resolved
ci/build_wheel.sh Outdated Show resolved Hide resolved
ci/build_wheel.sh Outdated Show resolved Hide resolved
ci/build_python.sh Outdated Show resolved Hide resolved
.github/ops-bot.yaml Show resolved Hide resolved
.github/workflows/pr.yaml Show resolved Hide resolved
.github/workflows/pr.yaml Show resolved Hide resolved
@nv-rliu nv-rliu requested a review from jameslamb October 31, 2024 20:10
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

I left some more suggestions for your consideration.

ci/build_python.sh Outdated Show resolved Hide resolved
ci/build_wheel.sh Outdated Show resolved Hide resolved
conda/recipes/nx-cugraph/meta.yaml Outdated Show resolved Hide resolved
ci/build_python.sh Outdated Show resolved Hide resolved
ci/build_python.sh Outdated Show resolved Hide resolved
@nv-rliu nv-rliu requested a review from jameslamb November 1, 2024 14:08
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

Just two more small suggestions... I'll apply them inline to kick off another build.

@KyleFromNVIDIA can you please re-review or dismiss your review?

ci/build_python.sh Outdated Show resolved Hide resolved
ci/build_wheel.sh Outdated Show resolved Hide resolved
@KyleFromNVIDIA KyleFromNVIDIA removed their request for review November 1, 2024 14:39
@jameslamb
Copy link
Member

Thanks for removing your name from reviewers @KyleFromNVIDIA , but your old review was still blocking:

image

I just dismissed it.

@nv-rliu
Copy link
Contributor Author

nv-rliu commented Nov 1, 2024

/merge

@rapids-bot rapids-bot bot merged commit 3b062b9 into rapidsai:branch-24.12 Nov 1, 2024
13 checks passed
@nv-rliu nv-rliu deleted the branch-24.12-pr-workflow branch November 1, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Introduces a breaking change ci conda Relates to conda packaging improvement Improves an existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.