-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Diagnostic renaming #121489
Merged
Merged
Diagnostic renaming #121489
Commits on Feb 27, 2024
-
Rename
Diagnostic
asDiagInner
.I started by changing it to `DiagData`, but that didn't feel right. `DiagInner` felt much better.
Configuration menu - View commit details
-
Copy full SHA for 6588f5b - Browse repository at this point
Copy the full SHA 6588f5bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 366536b - Browse repository at this point
Copy the full SHA 366536bView commit details -
Rename
SubDiagnostic
asSubdiag
.Note the change of the `D` to `d`, to match all the other names that have `Subdiag` in them, such as `SubdiagnosticMessage` and `derive(Subdiagnostic)`.
Configuration menu - View commit details
-
Copy full SHA for 4e1f9bd - Browse repository at this point
Copy the full SHA 4e1f9bdView commit details -
Rename
DiagnosticBuilder
asDiag
.Much better! Note that this involves renaming (and updating the value of) `DIAGNOSTIC_BUILDER` in clippy.
Configuration menu - View commit details
-
Copy full SHA for 899cb40 - Browse repository at this point
Copy the full SHA 899cb40View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8199632 - Browse repository at this point
Copy the full SHA 8199632View commit details -
Configuration menu - View commit details
-
Copy full SHA for d0ef108 - Browse repository at this point
Copy the full SHA d0ef108View commit details -
Configuration menu - View commit details
-
Copy full SHA for e3c19a2 - Browse repository at this point
Copy the full SHA e3c19a2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6280a8c - Browse repository at this point
Copy the full SHA 6280a8cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9b3520e - Browse repository at this point
Copy the full SHA 9b3520eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8f3b007 - Browse repository at this point
Copy the full SHA 8f3b007View commit details
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.