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

Fix renovate PR of rimraf 5.x #1228

Closed
wants to merge 2 commits into from

Conversation

wpoust
Copy link

@wpoust wpoust commented Jul 16, 2023

@pubkey

This PR contains:

  • Fixed the renovate PR to update to rimraf 5.x.

Describe the problem you have without this PR

The renovate PR for rimraf 5.x did not include changes for necessary for rimraf 5.x. Specifically, rimraf no longer uses a default export. In rimraf v4.x, the function returns a Promise. So, util.promisify is not needed.

@wpoust
Copy link
Author

wpoust commented Jul 17, 2023

The root cause of base failing is the browserify does not properly parse files in a glob that contain a class with private methods. I am not familiar with browserify. I'm looking for fixes for browserify parsing.

@pubkey
Copy link
Owner

pubkey commented Aug 18, 2023

Fixed here #1243

@pubkey pubkey closed this Aug 18, 2023
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