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

Implement a way to opt-out from yew lints #2360

Closed
voidpumpkin opened this issue Jan 11, 2022 · 1 comment · Fixed by #2882
Closed

Implement a way to opt-out from yew lints #2360

voidpumpkin opened this issue Jan 11, 2022 · 1 comment · Fixed by #2882
Labels
A-yew-macro Area: The yew-macro crate ergonomics feature-request A feature request

Comments

@voidpumpkin
Copy link
Member

voidpumpkin commented Jan 11, 2022 •
edited

Yew has some lints https://yew.rs/docs/concepts/html/introduction#lints
They were introduced in #1748 as part of #1334
But they have no way to opt-out #1748 (comment)
@WorldSEnder WorldSEnder added feature-request A feature request ergonomics A-yew-macro Area: The yew-macro crate labels May 8, 2022
@hamza1311 hamza1311 mentioned this issue Sep 24, 2022
Merged
2 tasks
@WorldSEnder WorldSEnder mentioned this issue Sep 25, 2022
Open
@WorldSEnder
Copy link
Member

WorldSEnder commented Sep 25, 2022 •
edited

#2882 made me look a bit deeper into custom lints, and there is ongoing work for more clippy/compiler integration. I've written a small user story in rust-marker/design#28. This will probably take a while, so the faster to merge, yew internal fix is still welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-yew-macro Area: The yew-macro crate ergonomics feature-request A feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make Yew lints opt-in hamza1311/yew
2 participants
@WorldSEnder @voidpumpkin