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

Replace name with [name] where appropriate #480

Closed
FWDekker opened this issue Oct 15, 2023 · 1 comment · Fixed by #487
Closed

Replace name with [name] where appropriate #480

FWDekker opened this issue Oct 15, 2023 · 1 comment · Fixed by #487
Assignees
@FWDekker
Labels
documentation Improvements or additions to documentation
Milestone
v3.0.0

Comments

@FWDekker
Copy link
Owner

Even if it would mean there's a lot of [name] in the docs, excessive links are preferred because then at least automatic refactoring tools also correctly update the docs.
@FWDekker FWDekker added the documentation Improvements or additions to documentation label Oct 15, 2023
@FWDekker FWDekker added this to the v3.0.0 milestone Oct 15, 2023
@FWDekker FWDekker self-assigned this Oct 15, 2023
@FWDekker
Copy link
Owner Author

Violating occurrences (and some false positives) in the docs can be found with the following regex: `(?!true|false|null|0|1|this|`|\\)[^`\s\=]*`

@FWDekker FWDekker changed the title Replace name with [name] where appropriate Replace ` name ` with [name] where appropriate Oct 22, 2023
@FWDekker FWDekker changed the title Replace ` name ` with [name] where appropriate Replace name with [name] where appropriate Oct 22, 2023
FWDekker added a commit that referenced this issue Oct 22, 2023
@FWDekker
Consistently use brackets to refer to classes
75f43a6 
Fixes #480.
@FWDekker FWDekker mentioned this issue Oct 22, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FWDekker FWDekker

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging a pull request may close this issue.

Docs and tests and stuff FWDekker/intellij-randomness
1 participant
@FWDekker