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

Replace glob with tinyglobby #311

Merged
merged 4 commits into from
Feb 12, 2025
Merged

Replace glob with tinyglobby #311

merged 4 commits into from
Feb 12, 2025

Conversation

lishaduck
Copy link
Contributor

@lishaduck lishaduck commented Nov 11, 2024

Part of #223.

Follow-up to #281.

Copy link

socket-security bot commented Dec 5, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

View full report↗︎

This lets us hit "full turbo" (where everything is skipped),
which we couldn't before.
We have to use `process.stdout.write` in compare
because we need to write the status afterward.
However, we never write a status in record, so the lines get scrunched up.
@lishaduck
Copy link
Contributor Author

lishaduck commented Feb 12, 2025

Hit rate limits, so I guess I'm blocking on one of #310 or a rerun?

@lishaduck lishaduck marked this pull request as ready for review February 12, 2025 20:10
@jfmengels jfmengels merged commit e5949d2 into jfmengels:main Feb 12, 2025
3 checks passed
@jfmengels
Copy link
Owner

jfmengels commented Feb 12, 2025

Hit rate limits, so I guess I'm blocking on one of #310 or a rerun?

Hopefully this is the last time!

@lishaduck lishaduck deleted the glob branch February 12, 2025 21:21
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