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

(Refactor) : Updated few dependencies related to : Webpack, React, Style-loader #185

Conversation

Abhijay007
Copy link
Contributor

@Abhijay007 Abhijay007 commented Mar 12, 2024

Summary of Changes

This pull request aims to update several dependencies to their latest versions to ensure compatibility, security, and leverage new features and enhancements. The following dependencies have been updated:

  1. react-d3-tree: Updated to version ^3.6.2
  2. react-icons: Updated to version ^5.0.1
  3. react-select: Updated to version ^5.8.0
  4. react-simplemde-editor: Updated to version ^5.2.0
  5. redux: Updated to version ^5.0.1
  6. redux-thunk: Updated to version ^3.1.0
  7. uuid: Updated to version ^9.0.1
  8. terser-webpack-plugin: Updated to version ^5.3.10
  9. webpack-cli: Updated to version ^5.1.4
  10. webpack-dev-server: Updated to version ^5.0.3
  11. style-loader: Updated to version ^3.3.4
  12. stylelint-config-standard-scss: Updated to version ^13.0.0

The some major changes in this pull request (PR) is related to the terser-webpack-plugin, webpack-cli, and webpack-dev-server. These were based on version 4, and there are some breaking changes in version 5. I addressed those changes. For this, I need to convert the following files:

configs/webpack.config.main.prod.babel.jsconfigs/webpack.config.main.prod.babel.cjs
configs/webpack.config.renderer.prod.babel.jsconfigs/webpack.config.renderer.prod.babel.cjs

There were a few errors around the imports, and upon investigation, I discovered that we need to change the extension of the .js file to either .mjs(Module JavaScript) or .cjs(Common JavaScript). I tried both, but using .mjs still throws some errors, so I opted for .cjs instead.

Also, some methods were updated in the newer version. For example, onBeforeSetupMiddleware() is no longer supported, so I replaced it with setupMiddlewares: ().

Other than this, there are just some formatting changes.

Related Issue

Closes #171

Testing:

  • Tested updated dependencies locally to ensure functionality remains intact.

    Used commands:

    • yarn test - all tests passed
    • yarn dev - no UI fails detected
    • yarn build - no build fails detected
    • yarn dev - no new console errors detected for any dependencies

Checklist

  • My code is well-documented, and I've updated relevant documentation.
  • I have tested these changes on my local environment.
  • I have reviewed and proofread my code and the changes.
  • The branch is up-to-date with the base branch.

Additional Context

Reviewer(s)

@lrasmus

Copy link
Contributor

@lrasmus lrasmus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good - thank you for the helpful documentation too about the necessary changes with dependency updates

@lrasmus lrasmus merged commit b44ab5d into StatTag:master Mar 14, 2024
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.

Refactor : Update Project Dependencies
2 participants