Skip to content
This repository was archived by the owner on Mar 10, 2024. It is now read-only.

fix: ignoreUnresolved #191

Merged
merged 1 commit into from
Nov 18, 2023
Merged

fix: ignoreUnresolved #191

merged 1 commit into from
Nov 18, 2023

Conversation

foucdeg
Copy link
Contributor

@foucdeg foucdeg commented Oct 29, 2023 •
edited
Loading

Fixes #190 by using the unresolved entry's toString (rather than its first part) to check for its presence inside the ignoreUnresolved index.

@foucdeg foucdeg changed the title fix ignoreUnresolvedIdx fix: ìgnoreUnresolved` is broken Oct 29, 2023
@foucdeg foucdeg changed the title fix: ìgnoreUnresolved` is broken fix: ignoreUnresolved Oct 29, 2023
@foucdeg foucdeg mentioned this pull request Oct 29, 2023
Closed
@smeijer smeijer merged commit 95a44c5 into smeijer:main Nov 18, 2023
@smeijer
Copy link
Owner

smeijer commented Nov 18, 2023

Thanks! Appreciated.

@smeijer
Copy link
Owner

smeijer commented Nov 18, 2023

🎉 This PR is included in version 1.31.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@smeijer
Copy link
Owner

smeijer commented Nov 18, 2023

@all-contributors please add @foucdeg for code

@allcontributors allcontributors bot mentioned this pull request Nov 18, 2023
Merged
@allcontributors AllContributors
Copy link
Contributor

@smeijer

I've put up a pull request to add @foucdeg! 🎉

@christian98 christian98 mentioned this pull request Nov 23, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@smeijer smeijer smeijer approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

-u option repetitively adding the same entry into ignoreUnresolved
2 participants
@foucdeg @smeijer