-
Notifications
You must be signed in to change notification settings - Fork 14k
[DO NOT MERGE] Add a crater lint for issue 145739 #149291
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
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
04bd0f3 to
46e68af
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
I have a crater permission by rust-lang/team#2038 but I'm not sure is this okay to run as I've never done it before 😅 |
Yes, it's fine (that's why r-a got crater permissions), I can assist (or at least try to assist) with the crater incantation if you need help. You can do smaller experimental batches for these too (before doing a full crater), e.g. pick top-50 or top-100 crates. |
|
The job Click to see the possible cause of the failure (guessed by this bot) |
Thanks! But this is not really a rust-analyzer side crate run, BTW 😅
Sounds like a good experiment to start with. But this branch currently fails on a CI lint for doc comments - doc requirements for new lints are stricter than I thought 😅 - though it succeeds on bootstraps and tests. Would it be okay to do crater run in current status? |
(I'm aware, this is a compiler side crater run :D) |
Crater runs only need to build, they don't need to pass all style checks or full tests. We can still give this a try, we can even just cherry-pick say 10 crates as a trial run, then check if crater can work. |
|
Only requirement is for a default try job to succeed: @bors try |
This comment has been minimized.
This comment has been minimized.
[DO NOT MERGE] Add a crater lint for #145739
|
@craterbot run mode=check-only crates=top-10 |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
Bumping priority because this is just a top-10 run to smoke-test the lint. @craterbot name=pr-149291 p=1 |
|
📝 Configuration of the ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🎉 Experiment
Footnotes
|
|
@craterbot check |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
I took some weird hack that expands the following macro call
into
and then look for those let stmt with
__issue_145739later in the late lint pass.It's because in early lint, we don't have much information other than AST, while in late lint, we have quite much information but not for AST 😂 So I have to leave some information about AST before executing late lint - macro expansion - and I think this would be okay as this lint will be used only for crater runs