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.

Sign up for GitHub

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

Jump to bottom

Fix format documentation to mention the Default trait #9808

Closed
orenbenkiki opened this issue Oct 11, 2013 · 1 comment · Fixed by #9938
Closed

Fix format documentation to mention the Default trait #9808

orenbenkiki opened this issue Oct 11, 2013 · 1 comment · Fixed by #9938
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@orenbenkiki
Copy link

The list of "Formatting traits" in libstd/fmt/mod.rs should include the Default trait, even though it is not associated with any format specifier.
@alexcrichton
Copy link
Member

Oh interesting, the Default trait isn't even mentioned at all!

@alexcrichton alexcrichton mentioned this issue Oct 18, 2013
Merged
bors added a commit that referenced this issue Oct 19, 2013
@bors
auto merge of #9938 : alexcrichton/rust/fmtdox, r=huonw
34a1e3d 
Closes #9865
Closes #9808
flip1995 pushed a commit to flip1995/rust that referenced this issue Dec 17, 2022
@bors
Auto merge of rust-lang#9920 - naosense:suppress_lint_in_const, r=xFr…
5d70746 
…ednet

add `suppress_restriction_lint_in_const` config

According to rust-lang#9808 , add a new lint `suppress_lint_in_const` to report even in const context. BTW, i am not good at naming either, if anyone have a better idea, i am happy to change it.

This PR is still in progress, so i keep it draft.

- \[x] Followed [lint naming conventions][lint_naming]
- \[x] Added passing UI tests (including committed `.stderr` file)
- \[x] `cargo test` passes locally
- \[x] Executed `cargo dev update_lints`
- \[x] Added lint documentation
- \[x] Run `cargo dev fmt`

changelog: Enhancement: [`indexing_slicing`]: add new config `suppress-restriction-lint-in-const` to enable restriction lints, even if the suggestion might not be applicable

r? `@xFrendet`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Document traits and Default about format! better alexcrichton/rust
2 participants
@alexcrichton @orenbenkiki