Skip to content

doc: add @[inherit_doc] to some local notation#11942

Closed
grunweg wants to merge 2 commits intomasterfrom
MR-inherit-local-notation
Closed

doc: add @[inherit_doc] to some local notation#11942
grunweg wants to merge 2 commits intomasterfrom
MR-inherit-local-notation

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Apr 6, 2024


My understanding is that because these are local notation, @eric-wieser's comment on zulip doesn't apply.

Open in Gitpod

@grunweg grunweg added documentation Improvements or additions to documentation awaiting-review labels Apr 6, 2024
Comment on lines +22 to +23
/-- Notation for two elements of a monoid are associated, i.e.
if one of them is another one multiplied by a unit on the right.-/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammer is wrong here. Maybe

Suggested change
/-- Notation for two elements of a monoid are associated, i.e.
if one of them is another one multiplied by a unit on the right.-/
/-- Notation for two elements of a monoid to be associated, i.e.
for one of them to be the other multiplied by a unit on the right. -/

or maybe just @[inherit_doc] as you already do in Algebra.Polynomial.Splits?

@YaelDillies YaelDillies added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jun 6, 2024
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Aug 12, 2024
@grunweg grunweg closed this Jan 12, 2025
@grunweg grunweg deleted the MR-inherit-local-notation branch January 12, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes. documentation Improvements or additions to documentation merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants