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

Cjs globby v13 esm #140

Closed

Conversation

frank-dspeed
Copy link

I Issue this PR only to discusse that with you in issue #138

Conclusion

because of this pull request that changes the api of sync as it now would return a Promise<string[]>

Two choices

  • we bundle globby v13 as cjs code i would at present not want that
  • we skip that and only merge the api upgrade and after that directly v7 esm only with globby 13 what do you think?

Updated to globby 13.1.1 returns now Promise<string[]> not string[] so this upgrade is not even possible. without new Major version 

## conclusion 
defer globby upgrade from 11 => 13 till ESM only version of del
Updated globby but that changes the output of the sync function so i bumped the minor version
Updated the typedef to reflect the minor api change
@frank-dspeed
Copy link
Author

i close this as this is for review only

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.

None yet

1 participant