Skip to content

Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior - #12144

Merged
bors merged 2 commits into
rust-lang:masterfrom
blyxyas:10283-postfix
Jan 20, 2024
Merged

Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior#12144
bors merged 2 commits into
rust-lang:masterfrom
blyxyas:10283-postfix

Conversation

@blyxyas

@blyxyas blyxyas commented Jan 14, 2024

Copy link
Copy Markdown
Member

Fixes #10283 (comment)

In the "/// Lint: " list on each configuration option, you have to end with a dot. If the lint list doesn't have a dot, the configuration won't have documentation.

This PR adds those missing dots in some of the configuration, thus also adding their documentation.

changelog: Fix bug where a lot of config documentation wasn't showing.
changelog: Fix typo in pub_underscore_fields_behavior (PublicallyExported -> PubliclyExported)

@rustbot

rustbot commented Jan 14, 2024

Copy link
Copy Markdown
Collaborator

r? @llogiq

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 14, 2024
@blyxyas blyxyas changed the title Add . to end of lint lists in configuration + Fix typo in Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior Jan 14, 2024
@llogiq

llogiq commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

Thank you!

@bors r+

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

📌 Commit 44f5d96 has been approved by llogiq

It is now in the queue for this repository.

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit 44f5d96 with merge 635736f...

bors added a commit that referenced this pull request Jan 20, 2024
Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior

Fixes #10283 (comment)

In the "/// Lint: " list on each configuration option, you have to end with a dot. If the lint list doesn't have a dot, the configuration won't have documentation.

This PR adds those missing dots in some of the configuration, thus also adding their documentation.

changelog: Fix bug where a lot of config documentation wasn't showing.
changelog: Fix typo in `pub_underscore_fields_behavior` (`PublicallyExported` -> `PubliclyExported`)
@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-action_test

@llogiq

llogiq commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

MacOS runner acting up.

@bors retry

bors added a commit that referenced this pull request Jan 20, 2024
Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior

Fixes #10283 (comment)

In the "/// Lint: " list on each configuration option, you have to end with a dot. If the lint list doesn't have a dot, the configuration won't have documentation.

This PR adds those missing dots in some of the configuration, thus also adding their documentation.

changelog: Fix bug where a lot of config documentation wasn't showing.
changelog: Fix typo in `pub_underscore_fields_behavior` (`PublicallyExported` -> `PubliclyExported`)
@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit 44f5d96 with merge e6f9367...

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-action_test

@blyxyas

blyxyas commented Jan 20, 2024

Copy link
Copy Markdown
Member Author

@bors retry

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit 44f5d96 with merge 9e9a229...

bors added a commit that referenced this pull request Jan 20, 2024
Add . to end of lint lists in configuration + Fix typo in pub_underscore_fields_behavior

Fixes #10283 (comment)

In the "/// Lint: " list on each configuration option, you have to end with a dot. If the lint list doesn't have a dot, the configuration won't have documentation.

This PR adds those missing dots in some of the configuration, thus also adding their documentation.

changelog: Fix bug where a lot of config documentation wasn't showing.
changelog: Fix typo in `pub_underscore_fields_behavior` (`PublicallyExported` -> `PubliclyExported`)
@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-action_test

@blyxyas

blyxyas commented Jan 20, 2024

Copy link
Copy Markdown
Member Author

@bors retry

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit 44f5d96 with merge fe3e682...

@bors

bors commented Jan 20, 2024

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing fe3e682 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants