Skip to content

Conversation

JoviDeCroock
Copy link
Member

@JoviDeCroock JoviDeCroock commented Aug 19, 2025

Yikes, my npm version might be too new judging from the lockfile changes

Copy link
Member

@rschristian rschristian left a comment

Choose a reason for hiding this comment

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

I'm on mobile so no lockfile diff, might be the first time it's being formatted (precommit hook)? I think I altered the filter on that recently, we might not've added or removed anything since.

@JoviDeCroock
Copy link
Member Author

Yes you are right, it's prettier diff, it does look like we might want to exclude the markdown content as it sometimes formats lines where we depends on indicator arrows in comments

Copy link
Member

@rschristian rschristian left a comment

Choose a reason for hiding this comment

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

The arrow issue I was aware of, though expecting people to use --no-verify when correcting might be unreasonable.

@JoviDeCroock JoviDeCroock merged commit 80818d4 into master Aug 22, 2025
5 checks passed
@JoviDeCroock JoviDeCroock deleted the bump-v11 branch August 22, 2025 06:14
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