Ben McCann
Ben McCann
> Note for testing - CI fails on this test file: test/monitor/count.test.js I think there might be a couple of other test failures as well. I looked at them briefly....
Personally I think it would look weird to put a space before all items except the first one. We ended up not using this (https://github.com/bombshell-dev/clack/issues/197#issuecomment-2489764465), but feel free to merge...
I just stumbled across this PR. I thought it'd be good to mention that nx, lerna, and pnpm have switched to tinyglobby
The ideal in my mind is previews that we can see and test against with each PR and then `main` gets deployed to production
We used this for an early version of [the Svelte CLI](https://npmjs.com/package/sv), but ended up going back to the standard prompts. A lot of the reason was unrelated to the design...
@pi0 just checking in to see if you might be able to give this a look. thanks!
Thanks for the update! I see now that you'd mentioned that on the PR targeting v2, but I'm afraid I'd forgotten Regarding the negated ignore feature, is there a project...
I certainly agree on the need to ignore specific patterns! That works in `tinyglobby` today. The thing that doesn't is if you want to unignore a sub-directory in public assets....
`@nuxt/kit`, `unimport`, and `vite` have all switched to `tinyglobby`. Nuxt exclusively relies on `tinyglobby` now except for `nitropack`. ``` my-nuxt-project$ pnpm why tinyglobby Legend: production dependency, optional only, dev only...
thanks!!