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.

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

Further clippy fixes #55

Merged
merged 7 commits into from
Feb 23, 2024
Merged

Conversation

waywardmonkeys
Copy link
Contributor

No description provided.

Using `clippy::doc_markdown` is useful for catching mistakes in
the documentation comments, but since `AutoCAD` and `PostScript`
look like identifiers, it warns about those needing backticks.

Using `clippy.toml`, we can suppress those warnings, making this
lint useful.
The lints have all been addressed, so this can be removed now.
@waywardmonkeys
Copy link
Contributor Author

This fixes all remaining clippy warnings after #42.

@brettfo This will conflict with another PR that I'd like to submit, so once this lands, I'll have to rebase another set of changes to be able to submit those.

@brettfo brettfo merged commit 6954d33 into ixmilia:main Feb 23, 2024
8 checks passed
@brettfo
Copy link
Member

brettfo commented Feb 23, 2024

Thank you for this! Apologies for me taking so long to address it.

@waywardmonkeys waywardmonkeys deleted the further-clippy-fixes branch February 23, 2024 17:53
@waywardmonkeys
Copy link
Contributor Author

Thanks for the merge! I'll try and remember what I'd planned on doing next...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants