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

Change dask and distributed branch to main [skip-ci] #7532

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

dantegd
Copy link
Member

@dantegd dantegd commented Mar 8, 2021

dask and distributed are changing their default branches name from master to main, this will break our dev environments and CI, this PR updates the required files.

distributed already merged the PR that does the change, dask will probably do the same very soon so a PR that updates both seems to be the best approach.

@dantegd dantegd added bug Something isn't working non-breaking Non-breaking change labels Mar 8, 2021
@dantegd dantegd changed the title Change dask and distributed branch to main Change dask and distributed branch to main [skip-ci] Mar 8, 2021
@jakirkham
Copy link
Member

Dask hasn't made the branch change yet. Asking for an update about that here ( dask/distributed#4495 (comment) )

@jrbourbeau
Copy link

Both dask and distributed have been switched over to main now 👍

@dantegd
Copy link
Member Author

dantegd commented Mar 8, 2021

rerun tests

@jakirkham
Copy link
Member

Thanks James! 😄

@jakirkham
Copy link
Member

Should this still be marked as draft?

@dantegd dantegd marked this pull request as ready for review March 8, 2021 20:31
@dantegd dantegd requested a review from a team as a code owner March 8, 2021 20:31
@kkraus14
Copy link
Collaborator

kkraus14 commented Mar 8, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 5d02c86 into rapidsai:branch-0.19 Mar 8, 2021
hyperbolic2346 pushed a commit to hyperbolic2346/cudf that referenced this pull request Mar 25, 2021
`dask` and `distributed` are changing their default branches name from `master` to `main`, this will break our dev environments and CI, this PR updates the required files. 

`distributed` already merged the PR that does the change, `dask` will probably do the same very soon so a PR that updates both seems to be the best approach.

Authors:
  - Dante Gama Dessavre (@dantegd)

Approvers:
  - Keith Kraus (@kkraus14)
  - AJ Schmidt (@ajschmidt8)

URL: rapidsai#7532
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants