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 GPU CI. #1449

Merged
merged 2 commits into from
Jan 4, 2021
Merged

Update GPU CI. #1449

merged 2 commits into from
Jan 4, 2021

Conversation

maleadt
Copy link
Collaborator

@maleadt maleadt commented Jan 4, 2021

Dummy changes to test status pushes from BuildKite. Now that it works, remove GitLab CI.

@CarloLucibello
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Jan 4, 2021
1449: Update GPU CI. r=CarloLucibello a=maleadt

Dummy changes to test status pushes from BuildKite. Now that it works, remove GitLab CI.

Co-authored-by: Tim Besard <[email protected]>
@maleadt
Copy link
Collaborator Author

maleadt commented Jan 4, 2021

Bors isn't required anymore now that BuildKite can do third-party PRs. Also note that it was only listening for gitlab statuses anyway: https://github.com/FluxML/Flux.jl/blob/ef1aa4927a06a70a2efa679a29ce3615c3e846eb/bors.toml. Either it should be updated to check the BuildKite status, all statusses, or removed altogether.

@DhairyaLGandhi
Copy link
Member

bors r-

@bors
Copy link
Contributor

bors bot commented Jan 4, 2021

Canceled.

@DhairyaLGandhi
Copy link
Member

Thanks so much!

@DhairyaLGandhi DhairyaLGandhi merged commit 16235e7 into master Jan 4, 2021
@CarloLucibello
Copy link
Member

so @DhairyaLGandhi you have to lift the github's restriction, merging is blocked here

@DhairyaLGandhi DhairyaLGandhi deleted the tb/ci branch January 8, 2021 16:57
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