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

Deploy time travel #5517

Merged
merged 2 commits into from
Apr 25, 2020
Merged

Deploy time travel #5517

merged 2 commits into from
Apr 25, 2020

Conversation

flip1995
Copy link
Member

Since not only commits to the master branch, but also tags and the beta branch are deployed, we have to be cautious which version of the deploy script is used. GHA always runs the workflow that is commited on the ref, that gets tested. For tagged commits. this is 6 weeks outdated workflows/scripts. To prevent this, this workflow first checks out the deploy.sh script, the website templates and all python scripts generating files for the website.

changelog: none

@flip1995 flip1995 requested a review from Manishearth April 23, 2020 18:59
@flip1995
Copy link
Member Author

@bors try

bors added a commit that referenced this pull request Apr 23, 2020
Deploy time travel

Since not only commits to the master branch, but also tags and the beta branch are deployed, we have to be cautious which version of the deploy script is used. GHA always runs the workflow that is commited on the `ref`, that gets tested. For tagged commits. this is 6 weeks outdated workflows/scripts. To prevent this, this workflow first checks out the deploy.sh script, the website templates and all python scripts generating files for the website.

changelog: none
@bors
Copy link
Contributor

bors commented Apr 23, 2020

⌛ Trying commit 37c5ae1 with merge ac43762...

@flip1995 flip1995 changed the title Deploy time travel [WIP] Deploy time travel Apr 23, 2020
@flip1995 flip1995 force-pushed the deploy_time_travel branch from 6d12a66 to 2de8115 Compare April 23, 2020 19:10
@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 23, 2020
@flip1995 flip1995 force-pushed the deploy_time_travel branch from 89481af to 7e579a6 Compare April 23, 2020 19:13
@flip1995 flip1995 force-pushed the deploy_time_travel branch from 7e579a6 to 0476e8b Compare April 23, 2020 19:15
@flip1995 flip1995 changed the title [WIP] Deploy time travel Deploy time travel Apr 23, 2020
@flip1995
Copy link
Member Author

@bors r=Manishearth

@bors
Copy link
Contributor

bors commented Apr 23, 2020

📌 Commit 0476e8b has been approved by Manishearth

bors added a commit that referenced this pull request Apr 23, 2020
Deploy time travel

Since not only commits to the master branch, but also tags and the beta branch are deployed, we have to be cautious which version of the deploy script is used. GHA always runs the workflow that is commited on the `ref`, that gets tested. For tagged commits. this is 6 weeks outdated workflows/scripts. To prevent this, this workflow first checks out the deploy.sh script, the website templates and all python scripts generating files for the website.

changelog: none
@bors
Copy link
Contributor

bors commented Apr 23, 2020

⌛ Testing commit 0476e8b with merge 2f33114...

@bors
Copy link
Contributor

bors commented Apr 23, 2020

💔 Test failed - checks-action_test

@flip1995 flip1995 added S-waiting-on-bors Status: The marked PR was approved and is only waiting bors and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Apr 23, 2020
@flip1995
Copy link
Member Author

@bors retry

flip1995 added a commit to flip1995/rust-clippy that referenced this pull request Apr 25, 2020
…hearth

Deploy time travel

Since not only commits to the master branch, but also tags and the beta branch are deployed, we have to be cautious which version of the deploy script is used. GHA always runs the workflow that is commited on the `ref`, that gets tested. For tagged commits. this is 6 weeks outdated workflows/scripts. To prevent this, this workflow first checks out the deploy.sh script, the website templates and all python scripts generating files for the website.

changelog: none
This was referenced Apr 25, 2020
bors added a commit that referenced this pull request Apr 25, 2020
Rollup of 5 pull requests

Successful merges:

 - #5408 (Downgrade match_bool to pedantic)
 - #5505 (Avoid running cargo+internal lints when not enabled)
 - #5516 (Add a note to the beta sections of release.md)
 - #5517 (Deploy time travel)
 - #5523 (Add lifetime test case for `new_ret_no_self`)

Failed merges:

r? @ghost

changelog: rollup
@bors bors merged commit abbc618 into rust-lang:master Apr 25, 2020
@flip1995 flip1995 deleted the deploy_time_travel branch May 4, 2020 15:29
flip1995 added a commit to flip1995/rust-clippy that referenced this pull request Jul 28, 2021
…ednet,flip1995

Switch CI to new metadata collection

r? `@xFrednet`

Things we have to keep in mind:

- This removes the template files and the scripts used for deployment from the checkout. This was added in rust-lang#5517. I don't think we ever needed those there. Not sure though.
- ~~As a result, we can't remove the python scripts yet. We have to wait until this hits a stable Clippy release.~~ I'll just break the next stable deploy and do it by hand once.
- This should be merged together with rust-lang#7279. Me and `@xFrednet` will coordinate the switch
- ...?

I still have to try out some things:

- [x] Is it worth caching? Yes
- [x] ~~Is it worth to do a release build?~~ Nope
- [x] Does it actually work? With a few changes, yes
- [ ] ...?

changelog: Clippy now uses a lint to generate its documentation 🎉
flip1995 added a commit to flip1995/rust-clippy that referenced this pull request Jul 28, 2021
…ednet,flip1995

Switch CI to new metadata collection

r? `@xFrednet`

Things we have to keep in mind:

- This removes the template files and the scripts used for deployment from the checkout. This was added in rust-lang#5517. I don't think we ever needed those there. Not sure though.
- ~~As a result, we can't remove the python scripts yet. We have to wait until this hits a stable Clippy release.~~ I'll just break the next stable deploy and do it by hand once.
- This should be merged together with rust-lang#7279. Me and `@xFrednet` will coordinate the switch
- ...?

I still have to try out some things:

- [x] Is it worth caching? Yes
- [x] ~~Is it worth to do a release build?~~ Nope
- [x] Does it actually work? With a few changes, yes
- [ ] ...?

changelog: Clippy now uses a lint to generate its documentation 🎉
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: The marked PR was approved and is only waiting bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants