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

Add more keywords (svg, parser, dom) to Cargo.toml #53

Closed
wants to merge 1 commit into from
Closed

Add more keywords (svg, parser, dom) to Cargo.toml #53

wants to merge 1 commit into from

Conversation

not-my-profile
Copy link

No description provided.

@IvanUkhov IvanUkhov closed this Oct 30, 2021
@not-my-profile
Copy link
Author

Do you mind commenting on why you closed this PR?

@IvanUkhov
Copy link
Member

Sorry. Sure! The idea is to avoid repetition. “svg” appears in the name; “parser” appears in the categories; and about “dom” I’m not sure why this might be necessary.

@not-my-profile
Copy link
Author

Thank you :)

I don't see the harm of repetition when it helps discoverability. The crates on crates.io can be browsed by keyword in which case neither the crate name nor the crate category are factored in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@not-my-profile @IvanUkhov