Skip to content

Incorporate changes from latest ignore crate - #19148

Merged
thecrypticace merged 1 commit into
mainfrom
bump-ignore-crate
Oct 20, 2025
Merged

Incorporate changes from latest ignore crate#19148
thecrypticace merged 1 commit into
mainfrom
bump-ignore-crate

Conversation

@thecrypticace

@thecrypticace thecrypticace commented Oct 18, 2025

Copy link
Copy Markdown
Contributor

This updates our internal fork of the ignore crate to incorporate changes from the latest release v0.4.24.

Aside: We should look into opening issues about the changes we had to make to see if they could be addressed in the crate itself so we can get rid of our fork.

@thecrypticace
thecrypticace marked this pull request as ready for review October 20, 2025 11:56
@thecrypticace
thecrypticace requested a review from a team as a code owner October 20, 2025 11:56

@RobinMalfait RobinMalfait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I tested it on some projects and everything still works as expected.

@RobinMalfait RobinMalfait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could add a changelog entry in case things break then we know why, but not sure if it's necesary.

@thecrypticace
thecrypticace merged commit 6c40814 into main Oct 20, 2025
21 checks passed
@thecrypticace
thecrypticace deleted the bump-ignore-crate branch October 20, 2025 13:49
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