-
Notifications
You must be signed in to change notification settings - Fork 296
Perform first rustc pull #1844
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
+5
−1
Merged
Perform first rustc pull #1844
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
c014f0a
Update README.md
kilavvy 97fa048
Rollup merge of #143176 - kilavvy:master, r=jieyouxu
dianqk 2b9fa3b
Rollup merge of #143187 - yotamofek:mailmap, r=jieyouxu
dianqk 53402a0
Rollup merge of #143190 - dianqk:new-method, r=oli-obk
dianqk 7a696e0
Rollup merge of #143195 - Kivooeo:tf17, r=tgross35
dianqk 0388fdb
Rollup merge of #143196 - Periodic1911:link_section, r=oli-obk
dianqk d64ae73
Rollup merge of #143199 - jieyouxu:short-ice, r=RalfJung
dianqk 791992a
Rollup merge of #143219 - zachs18:patch-5, r=tgross35
dianqk 51f0bad
Auto merge of #143233 - dianqk:rollup-lcx3278, r=dianqk
bors 1ab351a
Auto merge of #143239 - GuillaumeGomez:subtree-update_cg_gcc_2025-06-…
bors 318e3dc
Auto merge of #141875 - nnethercote:ByteSymbol, r=petrochenkov
bors d246a8c
Auto merge of #142921 - JonathanBrouwer:rustc_attributes_parser, r=ol…
bors 54d399e
Auto merge of #142030 - oli-obk:wfck-less-hir, r=compiler-errors
bors fa7ac71
Auto merge of #143214 - camsteffen:remove-let-chains-feature, r=est31
bors 0545c25
Auto merge of #142910 - yotamofek:pr/rustdoc/markdown-lazy-to-string,…
bors 2c2c768
Auto merge of #142890 - kornelski:unused-var-debug, r=saethlin
bors a73bceb
Add config files for `rustc-josh-sync`
Kobzol 6fc1814
Prepare for merging from rust-lang/rust
Kobzol 4571e48
Merge ref '1b61d43bdbf8' from rust-lang/rust
Kobzol File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
org = "rust-lang" | ||
repo = "stdarch" | ||
path = "library/stdarch" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
1b61d43bdbf875183b1f436302d62ff93f9a6bba | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ideally CI would use this to determine the Rust version to test, that'd made it stable under rustc repo changes... but that's a future TODO I think :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an interesting idea. It would mean that if there is a breaking change in rustc, we would only recognize it during the next pull (instead of recognizing it the next day when nightly is bumped). So we should combine this technique with having CI that will do an automatic pull every night to give us faster feedback when r-l/r changes break something in stdarch. I assume that's what miri does, right?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specifically we have a nightly CI job that starts with:
So we test against the latest rustc every night, and we get a Zulip message if that job fails. But even if rustc breaks, that doesn't affect unrelated PRs. If CI fails in a PR, that's always the fault of the PR because everything is pinned. IMO that is a quite important property to have.
This is orthogonal to the automatic pull, though we do both in the same CI cronjob.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is nice, yeah. I'll take a look at it once we have rustc-josh-sync for stdarch up & running.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to explain any questions about our CI setup that might come up. :) We spent quite a bit of effort on it over the years and I think it has served us well.