-
Notifications
You must be signed in to change notification settings - Fork 54
chore: switch to tinyglobby #1958
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
base: master
Are you sure you want to change the base?
Conversation
c9454d1 to
678b2bc
Compare
|
@prklm10 I just rebased this to address the merge conflicts. Any chance you could give it a look? It's a bit cumbersome to keep up-to-date |
cc8d4b6 to
22fdb4a
Compare
|
@prklm10 @pankaj443 @ninadbstack I'm terribly sorry to bother you, but would I be able to get you to take a look at this PR? I've been trying to keep it updated and it's a bit hard to keep it free of merge conflict |
22fdb4a to
3adebe9
Compare
|
@prklm10 @pankaj443 @ninadbstack please any chance that you could look at this? It's a pretty small straightforward PR |
| "image-size": "^1.0.0" | ||
| "image-size": "^1.0.0", | ||
| "tinyglobby": "^0.2.15" | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does remove a bit of the fast-glob (for good reasons - https://superchupu.dev/tinyglobby) so reduces the runtime size, so good imo. Fellas, let the tests run to give @benmccann feedback, so we can speed up install a bit, speeding up CI's for percy-cli users.
|
This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
not stale |
|
This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
Not stale |
|
This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
@prklm10 @pankaj443 @ninadbstack any chance this PR could get a review so I don't have to keep marking it as not stale? |
|
This PR is stale because it has been open for more than 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
Not stale |
https://npmgraph.js.org/?q=fast-glob - 17 dependencies
https://npmgraph.js.org/?q=tinyglobby - 2 dependencies