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 typo in test comment #17551

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

Fix typo in test comment #17551

wants to merge 1 commit into from

Conversation

mercyg
Copy link

@mercyg mercyg commented Jul 31, 2023

While reading this pr #17214 I noticed that there is a typo in the comment section for the index.js file

Summary

🤖 Generated by Copilot at b5a5d47

Fix a typo in a comment in test/configCases/code-generation/import-export-format/index.js. The comment clarifies the code generation logic for import and export statements.

Details

🤖 Generated by Copilot at b5a5d47

  • Fix a typo in a comment about import and export syntax (link)

@linux-foundation-easycla
Copy link

CLA Missing ID CLA Not Signed

@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)

Copy link
Member

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

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

@mercyg Can you accept CLA?

@mercyg
Copy link
Author

mercyg commented Aug 14, 2023

@mercyg Can you accept CLA?

@snitin315 I just signed the CLA

@snitin315
Copy link
Member

/easycla

@mercyg
Copy link
Author

mercyg commented Oct 23, 2023

@snitin315 can you please merge this

@mercyg mercyg requested a review from snitin315 December 5, 2023 02:38
@snitin315 snitin315 closed this Dec 5, 2023
@snitin315 snitin315 reopened this Dec 5, 2023
@snitin315
Copy link
Member

@mercyg CLA check is still failing, please make sure you sign CLA via the same email you commit your changes.

@mercyg
Copy link
Author

mercyg commented Dec 15, 2023

@snitin315 just signed again

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

Successfully merging this pull request may close these issues.

None yet

4 participants