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

update dev env link #15401

Closed
wants to merge 110 commits into from
Closed

Conversation

raybellwaves
Copy link
Contributor

@raybellwaves raybellwaves commented Mar 27, 2024

Going through the contributing document (https://github.com/rapidsai/cudf/blob/branch-24.06/CONTRIBUTING.md#create-the-build-environment) and thought i'd update the env to point to the cuda-120 file as I assume it's the most recent version and Is used more often. That said, i'm guessing the cuda-118 file exists to build against different cuda versions and it may have been chosen intensionally.

Apologies for small PR's i'm creating. I understand they take time to review. I'm not expecting and fast reviews and don't mind if they are closed for being out of scope. I just see small ways to increase the new user experience.

raydouglass and others added 30 commits March 30, 2020 11:03
Merge pull request rapidsai#5690 from ajschmidt8/phase2
[skip ci] Update master references for main branch
[RELEASE] Re-release v0.15 cudf [skip-ci]
[RELEASE] v0.18.2 `cudf` release [skip-ci]
Copy link

copy-pr-bot bot commented Mar 27, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@vyasr vyasr changed the base branch from main to branch-24.06 April 4, 2024 19:47
@vyasr vyasr requested review from a team as code owners April 4, 2024 19:47
@vyasr vyasr requested review from bdice, mroeschke and nvdbaranec April 4, 2024 19:47
@vyasr
Copy link
Contributor

vyasr commented Apr 4, 2024

Thanks for the contribution @raybellwaves! I agree that this would be a good change. We do our development off of release-versioned branches, so this should be based on branch-24.06 instead of main (confusing, I know). Could you please rebase your changes and push? Then we can get CI running here.

@raybellwaves
Copy link
Contributor Author

Thanks for the ping @vyasr . I opted to open #15467 as it's cleaner. Hope you don't mind

@vyasr
Copy link
Contributor

vyasr commented Apr 4, 2024

Not a problem, thanks!

rapids-bot bot pushed a commit that referenced this pull request Apr 13, 2024
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.

7 participants