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

[Dependency] Webpack to v5, istanbul removed #675

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thepiyush-303
Copy link

…ved istanbul-instrument-loader

@micpap25

I have updated webpack version from webpack 4.34.0 to webpack 5.93.0 , all the required packages and Removed the abandoned package istanbul-instrument-loader.

Hope it Helps.

Issue Submitty/Submitty#10641

@micpap25
Copy link
Contributor

@thepiyush-303 Please uncommit your dist files per the README; thank you!

@micpap25
Copy link
Contributor

@thepiyush-303 dist files are still listen in the files changed

@micpap25 micpap25 changed the title updated webpack and other required dependencies and successfully remo… [Dependency] Webpack to v5, other packages bumped, istanbul removed Jul 17, 2024
@micpap25 micpap25 changed the title [Dependency] Webpack to v5, other packages bumped, istanbul removed [Dependency] Webpack to v5, istanbul removed Jul 17, 2024
@micpap25
Copy link
Contributor

@thepiyush-303 your PR seems to be breaking, could you please address this?

@thepiyush-303
Copy link
Author

@micpap25
How can i fix these tests?

@micpap25
Copy link
Contributor

@thepiyush-303 the tests confirm that NPM can install and build the repository. So something is incorrect with your package.json configuration. Please confirm that it correctly installs on your machine before committing, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Work in Progress
Development

Successfully merging this pull request may close these issues.

2 participants