Skip to content

Remove unused code for download button #916

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

Merged

Conversation

jtgeibel
Copy link
Member

The download button has not been available since pre rust-1.0. The button was originally commented out in #15 and #18.

@jtgeibel jtgeibel force-pushed the remove-download-from-frontend branch from 396c068 to ff7f12a Compare July 26, 2017 04:14
@carols10cents
Copy link
Member

bors r+

@carols10cents
Copy link
Member

hmmm

The button was originally removed in rust-lang#15 and rust-lang#18
@carols10cents carols10cents force-pushed the remove-download-from-frontend branch from ff7f12a to 1aecb45 Compare July 29, 2017 00:34
@carols10cents
Copy link
Member

bors r+

@carols10cents
Copy link
Member

bors-voyager r+

@jtgeibel
Copy link
Member Author

Hey, looks awesome! I read about bors-ng recently. So it looks like this is testing on travis-ci now and will be automatically merged if the build passes, right?

@carols10cents
Copy link
Member

That's the theory :) Still working out a few kinks-- I deployed an instance of bors-ng to https://bors-voyager.herokuapp.com/ and I don't think i have it right yet...

@carols10cents
Copy link
Member

bors r+

bors-voyager bot added a commit that referenced this pull request Jul 29, 2017
916: Remove unused code for download button r=carols10cents

The download button has not been available since pre rust-1.0.  The button was originally commented out in #15 and #18.
@carols10cents
Copy link
Member

THERE WE GO!

@jtgeibel
Copy link
Member Author

It looks like it started a build for staging and staging.tmp. The staging.tmp branch doesn't exist but it looks like staging is still building fine.

@bors-voyager
Copy link
Contributor

bors-voyager bot commented Jul 29, 2017

Build succeeded

@bors-voyager bors-voyager bot merged commit 1aecb45 into rust-lang:master Jul 29, 2017
@jtgeibel
Copy link
Member Author

Yay, awesome!

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.

4 participants