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

add fs/promises to browser import ignore list #365

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

night
Copy link
Contributor

@night night commented Feb 18, 2025

With the latest release Webpack builds fail to compile as fs/promises is used instead of fs in one of the async imports. This PR adds fs/promises to the list of ignored browser imports

@harrisiirak
Copy link
Owner

Hi @night!

Did you check the latest (5.0.1) release? This removed fs and fs/promise from module level imports and these built-in modules are only imported dynamically in CronFileParser when it's actually being used.

If this doesn't solve the issue, I can consider merging this PR.

Best regards

@night
Copy link
Contributor Author

night commented Feb 18, 2025

This is after dependabot's upgrade to 5.0.1 from 4.9.0 in our repo. Webpack generally checks all imports, so this makes sense.

@harrisiirak harrisiirak self-requested a review February 18, 2025 15:25
@harrisiirak harrisiirak self-assigned this Feb 18, 2025
@harrisiirak harrisiirak merged commit fe6b90e into harrisiirak:master Feb 18, 2025
6 checks passed
@harrisiirak
Copy link
Owner

Merged and published a new version.

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