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

Add fuzzy matching for licenses search to add-icon-data script #10593

Merged
merged 3 commits into from
Mar 8, 2024

Conversation

mondeja
Copy link
Member

@mondeja mondeja commented Mar 8, 2024

Also:

  • Change usages of Boolean(string) by string.length comparisons because is more readable and the performance is the same.
  • Use trim() method validating aliases.

Contributes to #10572

@mondeja mondeja added the meta Issues or pull requests regarding the project or repository itself label Mar 8, 2024
@mondeja mondeja requested a review from LitoMore March 8, 2024 18:17
@github-actions github-actions bot removed the meta Issues or pull requests regarding the project or repository itself label Mar 8, 2024
@mondeja mondeja added meta Issues or pull requests regarding the project or repository itself dependencies Pull requests that update a dependency file labels Mar 8, 2024
@mondeja mondeja merged commit ce2f63e into simple-icons:develop Mar 8, 2024
3 checks passed
@mondeja mondeja deleted the fuzzy-add-icon-data-licenses branch March 8, 2024 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file meta Issues or pull requests regarding the project or repository itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants