Add identifiers to attributes.md and its subchapters #1560

chorman0773 · 2024-08-06T17:15:43Z

This adds identifiers to the attributes.md file and its subchapters, being:

attributes/codegen.md
attributes/debugging.md
attributes/derive.md
attributes/diagnostics.md
attributes/limits.md
attributes/testing.md
attributes/type_system.md

ehuss · 2024-09-04T20:19:37Z

src/attributes.md

 > &nbsp;&nbsp; &nbsp;&nbsp; [_DelimTokenTree_]\
 > &nbsp;&nbsp; | `=` [_Expression_]

+r[attributes.general]


Here and all elsewhere.

Suggested change

r[attributes.general]

r[attributes.intro]

(As an aside, I wonder if definition might fit these kinds of paragraphs better. We should probably define our common terms so that we understand when to use them appropriately.)

ehuss · 2024-09-04T20:21:31Z

src/attributes/codegen.md


 ### Behavior

+r[attributes.codegen.track_caller.behaviour]


Here and elsewhere

Suggested change

r[attributes.codegen.track_caller.behaviour]

r[attributes.codegen.track_caller.behavior]

ehuss · 2024-09-04T20:21:53Z

src/attributes/codegen.md


+r[attributes.codegen.track_caller.limits]
+
+r[attributes.codegent.track_caller.hint]


Please be careful about spelling.

Suggested change

r[attributes.codegent.track_caller.hint]

r[attributes.codegen.track_caller.hint]

ehuss · 2024-09-04T20:22:13Z

src/attributes/codegen.md


 ### The `inline` attribute

+r[attributes.codgen.inline]


Suggested change

r[attributes.codgen.inline]

r[attributes.codegen.inline]

ehuss · 2024-09-04T20:22:32Z

src/attributes/diagnostics.md

 warnings on usage of `#[deprecated]` items. `rustdoc` will show item
 deprecation, including the `since` version and `note`, if available.

+r[attributes.diagnostics.deprectead.syntax]


Suggested change

r[attributes.diagnostics.deprectead.syntax]

r[attributes.diagnostics.deprecated.syntax]

ehuss · 2024-09-04T22:08:16Z

src/attributes/debugger.md


+r[attributes.debugger.debugger_visualizer.natvis]
+
+r[attributes.debugger.debugger_visualizer.natvis-general]


Any particular reason to use - and not . for nativis-general?

ehuss · 2024-09-04T22:08:52Z

src/attributes/derive.md

+r[attributes.derive]
+
+
+r[attributes.derived.general]


Suggested change

r[attributes.derived.general]

r[attributes.derive.general]

ehuss · 2024-09-04T22:14:13Z

src/attributes/diagnostics.md

 5i32.use_me();
 ```

+r[attributes.diagnostics.must_use.impl-function]


To distinguish from other non-trait impl, maybe?

Suggested change

r[attributes.diagnostics.must_use.impl-function]

r[attributes.diagnostics.must_use.trait-impl-function]

ehuss · 2024-09-04T22:19:10Z

src/attributes/diagnostics.md


 ## The `diagnostic` tool attribute namespace

+r[attributes.diagnostics.namespace]


Hm, I'm on the fence here, since the namespace is singular diagnostic. I realize this is in the "diagnostics" section, and being inconsistent with the rest of the section could be more trouble. I'm just a little concerned about seeing diagnostics.namespace could confuse a reader, since it is a subtle distinction. WDYT?

I'm thinking the section should be changed to diagnostic because I've been otherwise favouring the singular form except for lists and when the construct is plural in the language.

ehuss · 2024-09-04T22:20:30Z

src/attributes/limits.md


 ## The `recursion_limit` attribute

+r[attributes.limits.recursion_limits]


Here and elsewhere.

Suggested change

r[attributes.limits.recursion_limits]

r[attributes.limits.recursion_limit]

chorman0773 · 2024-09-12T14:56:39Z

I need to modify the identifiers to be more consistent with the other chapters, and the guidelines I wrote.

@rustbot modify-labels +S-waiting-on-author -S-waiting-on-review

traviscross · 2024-10-17T23:41:20Z

@rustbot labels +S-waiting-on-author -S-waiting-on-review

The CI is failing on this one.

chorman0773 · 2024-10-21T16:27:02Z

@rustbot labels +S-waiting-on-review -S-waiting-on-author

This is just for navigation, these aren't specific rules.

ehuss

Thanks!

These are some misc rule name changes. Some of them are to give the rules more specific names (indicating what they do, instead of using generic terms). The `allowed-positions` name is intended to be used for all attributes to have a specific rule that defines where an attribute may be used. It is not consistently used, yet. Also, some of the paragraphs mix "where it goes" with "what it does" that should probably be teased apart so that we can label those independently. I expect us to do some more cleanup to bring some consistency to the attribute rules since there are a lot of attributes, and they generally have the same requirements of what needs to be specified.

This is keeping in line with using plurals throughout the rest of the naming convention. This also differentiates it from the `diagnostic` namespace. The `diagnostic` namespace remains singular, and is not nested with the plural `diagnostics` just to avoid the repetition.

Update books ## rust-lang/edition-guide 2 commits in 2d482e203eb6d6e353814cf1415c5f94e590b9e0..915f9b319c2823f310430ecdecd86264a7870d7e 2024-11-06 07:23:07 UTC to 2024-11-05 09:03:41 UTC - 2021: Update for raw lifetimes (rust-lang/edition-guide#330) - CI: Switch to merge queue (rust-lang/edition-guide#333) ## rust-lang/nomicon 1 commits in 456b904f791751892b01282fd2757904993c4c26..eac89a3cbe6c4714e5029ae8b5a1c556fd4e8c42 2024-11-16 14:05:28 UTC to 2024-11-16 14:05:28 UTC - Fix typo in what-unsafe-does (rust-lang/nomicon#469) ## rust-lang/reference 10 commits in da0f6dad767670da0e8cd5af8a7090db3272f626..41ccb0e6478305401dad92e8fd3d04a4304edb4c 2024-11-15 21:45:16 UTC to 2024-11-05 21:46:30 UTC - Add identifiers to attributes.md and its subchapters (rust-lang/reference#1560) - Fix 2 typos (rust-lang/reference#1674) - Add examples to clarify the casting rules (rust-lang/reference#686) - Explaining how to link mixed C/Rust binaries. (rust-lang/reference#838) - Add Arm64EC to inline-assembly documentation (rust-lang/reference#1653) - Add three more WebAssembly features to the list of accepted ones (rust-lang/reference#1638) - Add s390x to inline-assembly documentation (rust-lang/reference#1643) - trait object constraint correction (rust-lang/reference#1670) - Update some "default" representation references (rust-lang/reference#1667) - Update lifetimes for pre-expansion validation (rust-lang/reference#1668) ## rust-lang/rustc-dev-guide 12 commits in 6a5accd..b679e71 2024-11-18 08:18:15 UTC to 2024-11-05 07:22:17 UTC - Mention `RUSTC_BOOTSTRAP` for misc testing (rust-lang/rustc-dev-guide#2136) - Document how to acquire `cdb.exe` (rust-lang/rustc-dev-guide#2137) - Document `max-llvm-major-version` directive (rust-lang/rustc-dev-guide#2129) - Document `exact-llvm-major-version` directive (rust-lang/rustc-dev-guide#2135) - Note Rustfmt for separate rust-analyzer directory (rust-lang/rustc-dev-guide#2134) - still accurate (rust-lang/rustc-dev-guide#2133) - typo (rust-lang/rustc-dev-guide#2132) - add valid date-check marker (rust-lang/rustc-dev-guide#2131) - Update parallel-rustc.md (rust-lang/rustc-dev-guide#1926) - Rename `{ignore,only}-debug` -> `{ignore,needs}-{rustc,std}-debug-assertions` (rust-lang/rustc-dev-guide#2101) - update const stability docs (rust-lang/rustc-dev-guide#2111) - Ask folks to use the Oxford comma in diagnostics (rust-lang/rustc-dev-guide#2093)

Rollup merge of rust-lang#133181 - rustbot:docs-update, r=ehuss Update books ## rust-lang/edition-guide 2 commits in 2d482e203eb6d6e353814cf1415c5f94e590b9e0..915f9b319c2823f310430ecdecd86264a7870d7e 2024-11-06 07:23:07 UTC to 2024-11-05 09:03:41 UTC - 2021: Update for raw lifetimes (rust-lang/edition-guide#330) - CI: Switch to merge queue (rust-lang/edition-guide#333) ## rust-lang/nomicon 1 commits in 456b904f791751892b01282fd2757904993c4c26..eac89a3cbe6c4714e5029ae8b5a1c556fd4e8c42 2024-11-16 14:05:28 UTC to 2024-11-16 14:05:28 UTC - Fix typo in what-unsafe-does (rust-lang/nomicon#469) ## rust-lang/reference 10 commits in da0f6dad767670da0e8cd5af8a7090db3272f626..41ccb0e6478305401dad92e8fd3d04a4304edb4c 2024-11-15 21:45:16 UTC to 2024-11-05 21:46:30 UTC - Add identifiers to attributes.md and its subchapters (rust-lang/reference#1560) - Fix 2 typos (rust-lang/reference#1674) - Add examples to clarify the casting rules (rust-lang/reference#686) - Explaining how to link mixed C/Rust binaries. (rust-lang/reference#838) - Add Arm64EC to inline-assembly documentation (rust-lang/reference#1653) - Add three more WebAssembly features to the list of accepted ones (rust-lang/reference#1638) - Add s390x to inline-assembly documentation (rust-lang/reference#1643) - trait object constraint correction (rust-lang/reference#1670) - Update some "default" representation references (rust-lang/reference#1667) - Update lifetimes for pre-expansion validation (rust-lang/reference#1668) ## rust-lang/rustc-dev-guide 12 commits in 6a5accd..b679e71 2024-11-18 08:18:15 UTC to 2024-11-05 07:22:17 UTC - Mention `RUSTC_BOOTSTRAP` for misc testing (rust-lang/rustc-dev-guide#2136) - Document how to acquire `cdb.exe` (rust-lang/rustc-dev-guide#2137) - Document `max-llvm-major-version` directive (rust-lang/rustc-dev-guide#2129) - Document `exact-llvm-major-version` directive (rust-lang/rustc-dev-guide#2135) - Note Rustfmt for separate rust-analyzer directory (rust-lang/rustc-dev-guide#2134) - still accurate (rust-lang/rustc-dev-guide#2133) - typo (rust-lang/rustc-dev-guide#2132) - add valid date-check marker (rust-lang/rustc-dev-guide#2131) - Update parallel-rustc.md (rust-lang/rustc-dev-guide#1926) - Rename `{ignore,only}-debug` -> `{ignore,needs}-{rustc,std}-debug-assertions` (rust-lang/rustc-dev-guide#2101) - update const stability docs (rust-lang/rustc-dev-guide#2111) - Ask folks to use the Oxford comma in diagnostics (rust-lang/rustc-dev-guide#2093)

chorman0773 added S-waiting-on-review Status: The marked PR is awaiting review from a maintainer T-spec Team: spec labels Aug 6, 2024

chorman0773 mentioned this pull request Aug 6, 2024

Tracking Issue: Adding mdbook-spec identifiers to reference chapters #1561

Closed

94 tasks

ehuss reviewed Sep 4, 2024

View reviewed changes

chorman0773 added S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. and removed S-waiting-on-review Status: The marked PR is awaiting review from a maintainer labels Sep 12, 2024

chorman0773 added 12 commits October 21, 2024 12:20

Add identifier syntax to attributes.md

173a829

Add identifier syntax to attributes.codegen

e5acdc2

Add identifier syntax to attributes.debugger

41fe211

Add identifier syntax to attributes.derive

3b6ebe1

Add identifier syntax to attributes.diagnostics

42fbf9b

Add identifier syntax to attributes.limits

8f980cb

Add identifier syntax to attributes.testing

0657657

Add identifier syntax to attributes.type_system

2b5d973

Fix style in modified chapters

043c99f

Appease link-checker

b1a26cc

Apply suggestions from PR review

aa93e94

Replace uses of constraint and restriction in attributes chapters.

d3ef277

chorman0773 force-pushed the spec-add-identifiers-attributes branch from e579b2a to d3ef277 Compare October 21, 2024 16:20

rustbot added S-waiting-on-review Status: The marked PR is awaiting review from a maintainer and removed S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. labels Oct 21, 2024

ehuss added 4 commits November 15, 2024 13:25

Fix various spelling and format errors

ccbc70d

Add back some newlines that were removed

e0f31a0

Remove some double blank lines

da65af0

Remove annotations from the attribute index

b62fbd2

This is just for navigation, these aren't specific rules.

ehuss approved these changes Nov 15, 2024

View reviewed changes

ehuss force-pushed the spec-add-identifiers-attributes branch from 94aa44f to 4ef5f5d Compare November 15, 2024 21:42

ehuss enabled auto-merge November 15, 2024 21:43

ehuss added this pull request to the merge queue Nov 15, 2024

Merged via the queue into rust-lang:master with commit 41ccb0e Nov 15, 2024
5 checks passed

rustbot mentioned this pull request Nov 18, 2024

Update books rust-lang/rust#133181

Merged

	r[attributes.codegen.track_caller.behaviour]
	r[attributes.codegen.track_caller.behavior]


		r[attributes.codegen.track_caller.limits]

		r[attributes.codegent.track_caller.hint]

	r[attributes.codegent.track_caller.hint]
	r[attributes.codegen.track_caller.hint]

	r[attributes.diagnostics.deprectead.syntax]
	r[attributes.diagnostics.deprecated.syntax]


		r[attributes.debugger.debugger_visualizer.natvis]

		r[attributes.debugger.debugger_visualizer.natvis-general]

	r[attributes.diagnostics.must_use.impl-function]
	r[attributes.diagnostics.must_use.trait-impl-function]


		## The `diagnostic` tool attribute namespace

		r[attributes.diagnostics.namespace]


		## The `recursion_limit` attribute

		r[attributes.limits.recursion_limits]

	r[attributes.limits.recursion_limits]
	r[attributes.limits.recursion_limit]

		r[attributes.derive]


		r[attributes.derived.general]

Add identifiers to attributes.md and its subchapters #1560

Add identifiers to attributes.md and its subchapters #1560

Uh oh!

Conversation

chorman0773 commented Aug 6, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chorman0773 commented Sep 12, 2024

Uh oh!

traviscross commented Oct 17, 2024

Uh oh!

chorman0773 commented Oct 21, 2024

Uh oh!

ehuss left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants