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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests for ModuleFilenameHelpers #17784

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mayankkamboj47
Copy link

@mayankkamboj47 mayankkamboj47 commented Nov 6, 2023

As per #16851, add tests for ModuleFilenameHelpers. This PR only adds tests for functions behaving as expected. Refer to #16851 for details on others

Summary

馃 Generated by Copilot at b2262eb

This pull request improves the documentation and testing of the webpack project. It fixes some formatting and style issues in _SETUP.md, CONTRIBUTING.md, and README.md, and adds a reference to SECURITY.md in the contribution guidelines. It also adds unit tests for the ModuleFilenameHelpers module in test/ModuleFilenameHelpers.test.js.

Details

馃 Generated by Copilot at b2262eb

  • Capitalize webpack in README.md and CONTRIBUTING.md for consistency and branding (link, link, link, link, link)
  • Fix broken link to concepts page in README.md (link)
  • Add Reporting Security Issues section from SECURITY.md to CONTRIBUTING.md to consolidate information for contributors (link)
  • Remove duplicate and extra line breaks in CONTRIBUTING.md for better readability (link, link)
  • Add missing code block for yarn lint command and remove duplicate code block in _SETUP.md (link, link)
  • Add unit tests for ModuleFilenameHelpers module in test/ModuleFilenameHelpers.test.js (link)

@webpack-bot
Copy link
Contributor

For maintainers only:

  • This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
  • This needs to be backported to webpack 4 (issue will be created when merged)

@mayankkamboj47
Copy link
Author

Could you please review this ?

@alexander-akait
Copy link
Member

@mayankkamboj47 Can you fix CI?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: On Hold
Development

Successfully merging this pull request may close these issues.

None yet

3 participants