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

[v14.x backport] deps: update to cjs-module-lexer@1.2.2 #39421

Closed

Conversation

guybedford
Copy link
Contributor

Backports #39402 for Node.js 14.x.

@nodejs/modules

@github-actions github-actions bot added needs-ci PRs that need a full CI run. v14.x labels Jul 17, 2021
@nodejs-github-bot
Copy link
Collaborator

PR-URL: nodejs#39402
Reviewed-By: Bradley Farias <bradley.meck@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@guybedford
Copy link
Contributor Author

Not sure what's going on with this CI run here, but this should still be good to merge I believe. Are there known issues with the 14 builds?

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@guybedford guybedford removed the needs-ci PRs that need a full CI run. label Jul 24, 2021
@guybedford
Copy link
Contributor Author

Landed in ac76cdb.

@guybedford guybedford closed this Jul 26, 2021
@guybedford guybedford deleted the cjs-module-lexer-update-14 branch July 26, 2021 12:23
@MylesBorins
Copy link
Member

@guybedford this landed a bit pre-maturely. The original change hasn't yet gone out in a 16.x release and we need it to be out on Current for at least 2 weeks before backporting. Generally I've avoided landing changes that are not yet ready for backport to avoid it accidentally going out pre-maturely in an LTS release

@guybedford
Copy link
Contributor Author

Thanks @MylesBorins I've reverted and reopened this in #39531.

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.

None yet

4 participants