refactor: Expose source/spans to Manifest for emitting lints#13593
Merged
bors merged 3 commits intorust-lang:masterfrom Mar 15, 2024
Merged
refactor: Expose source/spans to Manifest for emitting lints#13593bors merged 3 commits intorust-lang:masterfrom
bors merged 3 commits intorust-lang:masterfrom
Conversation
This does nothing on its own. This is meant to short-circuit some of my refactorings so Muscraft can start on their work on adding lints while I work to move out existing warnings into being able to be converted to lints.
Collaborator
rustbot
added
A-manifest
Muscraft
approved these changes
Mar 15, 2024
Member
Muscraft
left a comment
Muscraft
left a comment
There was a problem hiding this comment.
This will be helpful for the diagnostic system!
Member
|
@bors r+ (Doing this before all the tests pass but I don't expect any failures) |
Contributor
bors
added
S-waiting-on-bors
Contributor
Member
Better! Thanks! |
Contributor
|
☀️ Test successful - checks-actions |
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Mar 16, 2024
Update cargo 6 commits in 7065f0ef4aa267a7455e1c478b5ccacb7baea59c..2fe739fcf16c5bf8c2064ab9d357f4a0e6c8539b 2024-03-12 13:25:15 +0000 to 2024-03-15 21:39:18 +0000 - feat: Add 'open-namespaces' feature (rust-lang/cargo#13591) - refactor: Expose source/spans to Manifest for emitting lints (rust-lang/cargo#13593) - feat(tree): Control `--charset` via auto-detecting config value (rust-lang/cargo#13337) - refactor(toml): Flatten manifest parsing (rust-lang/cargo#13589) - fix: strip feature dep when dep is dev dep (rust-lang/cargo#13518) - fix(ci): bump check error when PR is behind master (rust-lang/cargo#13581) r? ghost
bors
added a commit
that referenced
this pull request
Mar 19, 2024
refactor(toml): Expose surce/spans for VirtualManifests ### What does this PR try to resolve? This is a follow up to #13593, expanding support from `Manifest` to `VirtualManifest` as well. This also does other clean up along the way in preparation for making a more explicit `resolve_toml` phase. ### How should we test and review this PR? ### Additional information
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
What does this PR try to resolve?
This is a follow up to #13589.
This does nothing on its own.
This is meant to short-circuit some of my refactorings so Muscraft can
start on their work on adding lints while I work to move out existing
warnings into being able to be converted to lints.
How should we test and review this PR?
This includes documentation changes suggested in #13589
Additional information