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.

Sign up for GitHub

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

Jump to bottom

doc: fix "the the" typo in README.md of cjs-module-lexer #37895

Merged
merged 1 commit into from
Mar 30, 2021
Merged

doc: fix "the the" typo in README.md of cjs-module-lexer #37895

merged 1 commit into from
Mar 30, 2021

Conversation

peZhmanParsaee
Copy link
Contributor

No description provided.

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Mar 24, 2021
@peZhmanParsaee peZhmanParsaee changed the title fix "the the" typo in README.md of cjs-module-lexer doc: fix "the the" typo in README.md of cjs-module-lexer Mar 24, 2021
@aduh95
Copy link
Contributor

aduh95 commented Mar 24, 2021

/cc @guybedford

aduh95
aduh95 requested changes Mar 25, 2021
View reviewed changes
Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

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

This change needs to be accepted upstream first (at https://github.com/guybedford/cjs-module-lexer) before it can land here.

@peZhmanParsaee peZhmanParsaee mentioned this pull request Mar 25, 2021
Closed
@peZhmanParsaee
Copy link
Contributor Author

This change needs to be accepted upstream first (at https://github.com/guybedford/cjs-module-lexer) before it can land here.

Hi @aduh95 , changes was applied on cjs-module-lexer repository. Would you reconsider your review? Tnx

@RaisinTen
Copy link
Contributor

@peZhmanParsaee We tend not to make changes to the parts in deps as these are updated with a PR when a new release of the dependency is available. You may check out #37712 to get an idea of how it has been done for cjs-module-lexer.

@aduh95
Copy link
Contributor

aduh95 commented Mar 26, 2021

Thanks a lot for taking the time to send the PR upstream. Guy has indicated in nodejs/cjs-module-lexer#48 (comment) that there's likely no upcoming release in the near future, so I suppose it is fine to land this here before it's officially released upstream. I'll try to take care of this early next week.

@peZhmanParsaee
Copy link
Contributor Author

@aduh95

Thanks a lot dude
Your help is much appreciated and has given much positive energy to me :)

@peZhmanParsaee @aduh95
deps: cjs-module-lexer: cherry-pick 22093e765f
d7b8638 
Original commit message:
	fix "the the" typo in README.md file

Refs: nodejs/cjs-module-lexer@22093e7

PR-URL: nodejs#37895
Reviewed-By: Pooja D P <Pooja.D.P@ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@aduh95 aduh95 force-pushed the doc-cjs-module-lexer-readme-typo branch from 45dc755 to d7b8638 Compare March 30, 2021 08:15
@aduh95 aduh95 merged commit d7b8638 into nodejs:master Mar 30, 2021
@aduh95
Copy link
Contributor

aduh95 commented Mar 30, 2021

Landed in d7b8638. Thanks for your contribution :)

MylesBorins pushed a commit that referenced this pull request Apr 4, 2021
@peZhmanParsaee @MylesBorins
deps: cjs-module-lexer: cherry-pick 22093e765f
913ec56 
Original commit message:
	fix "the the" typo in README.md file

Refs: nodejs/cjs-module-lexer@22093e7

PR-URL: #37895
Reviewed-By: Pooja D P <Pooja.D.P@ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Apr 4, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aduh95 aduh95 aduh95 approved these changes

@marsonya marsonya marsonya approved these changes

@HarshithaKP HarshithaKP HarshithaKP approved these changes

@PoojaDurgad PoojaDurgad PoojaDurgad approved these changes

Assignees
No one assigned
Labels
needs-ci PRs that need a full CI run.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@peZhmanParsaee @aduh95 @RaisinTen @marsonya @HarshithaKP @PoojaDurgad @targos @nodejs-github-bot