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

Pin Node 18 to 18.2.0 to avoid npm issue #576

Merged
merged 1 commit into from
Jun 2, 2022
Merged

Pin Node 18 to 18.2.0 to avoid npm issue #576

merged 1 commit into from
Jun 2, 2022

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Jun 2, 2022

See also: #552 (comment)

@daviwil daviwil mentioned this pull request Jun 2, 2022
Merged
@azure-pipelines
Copy link

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/576/

@daviwil
Copy link
Contributor Author

daviwil commented Jun 2, 2022

I'll send a PR for cadl-azure repo next.

@daviwil daviwil merged commit dee20cf into microsoft:main Jun 2, 2022
@daviwil daviwil deleted the fix-node-18 branch June 2, 2022 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timotheeguerin timotheeguerin timotheeguerin approved these changes

@bterlson bterlson Awaiting requested review from bterlson bterlson is a code owner

@nguerrera nguerrera Awaiting requested review from nguerrera

@markcowl markcowl Awaiting requested review from markcowl markcowl is a code owner

@allenjzhang allenjzhang Awaiting requested review from allenjzhang allenjzhang is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daviwil @timotheeguerin