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.

Sign up for GitHub

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

Jump to bottom

Build packages using mambabuild #486

Merged
merged 1 commit into from
May 24, 2022
Merged

Build packages using mambabuild #486

merged 1 commit into from
May 24, 2022

Conversation

jjacobelli
Copy link
Contributor

@jjacobelli jjacobelli commented Feb 1, 2022 •
edited
Loading

Use mambabuild to build conda packages. This should speed up the builds and help to debug conda conflict issues

@jjacobelli jjacobelli added 5 - DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 1, 2022
@jjacobelli jjacobelli requested a review from a team February 1, 2022 15:20
@jjacobelli jjacobelli self-assigned this Feb 1, 2022
@github-actions github-actions bot added the gpuCI Related to Continuous Integration / Automation label Feb 1, 2022
@jjacobelli jjacobelli marked this pull request as draft February 2, 2022 08:32
@jjacobelli jjacobelli removed the 5 - DO NOT MERGE Hold off on merging; see PR for details label Feb 2, 2022
@jjacobelli jjacobelli marked this pull request as ready for review February 9, 2022 15:44
@jjacobelli jjacobelli marked this pull request as draft February 9, 2022 16:24
@github-actions
Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

@harrism
Copy link
Member

harrism commented Mar 14, 2022

@Ethyling what's needed to get this merged?

@jjacobelli
Copy link
Contributor Author

@Ethyling what's needed to get this merged?

@harrism We need to use conda compilers before merging this. Related PR: #495
Moved to 22.06 for now

@jjacobelli jjacobelli changed the base branch from branch-22.04 to branch-22.06 March 24, 2022 09:24
@github-actions
Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

@jjacobelli
Copy link
Contributor Author

rerun tests

@jjacobelli jjacobelli marked this pull request as ready for review May 23, 2022 14:08
@jjacobelli
Copy link
Contributor Author

rerun tests

@jjacobelli
Build packages using mambabuild
faec47c 
Signed-off-by: Jordan Jacobelli <jjacobelli@nvidia.com>
@jjacobelli
Copy link
Contributor Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 700ac12 into rapidsai:branch-22.06 May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajschmidt8 ajschmidt8 ajschmidt8 approved these changes

Assignees

@jjacobelli jjacobelli

Labels
gpuCI Related to Continuous Integration / Automation improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jjacobelli @harrism @ajschmidt8