Skip to content

Support node v10, v12, or v14 for amphtml-validator#25208

Merged
rsimha merged 3 commits intoampproject:masterfrom
estherkim:node-10-for-validator
Oct 22, 2019
Merged

Support node v10, v12, or v14 for amphtml-validator#25208
rsimha merged 3 commits intoampproject:masterfrom
estherkim:node-10-for-validator

Conversation

@estherkim
Copy link
Copy Markdown
Collaborator

No description provided.

@estherkim estherkim requested a review from rsimha October 22, 2019 22:12
@estherkim estherkim changed the title Support node v10 or above for amphtml-validator Support node v10, v12, or v14 for amphtml-validator Oct 22, 2019
Copy link
Copy Markdown
Member

@twifkak twifkak left a comment

Choose a reason for hiding this comment

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

LGTM on behalf of @ampproject/wg-caching.

cc @cpapazian

@rsimha
Copy link
Copy Markdown
Contributor

rsimha commented Oct 22, 2019

Force merging this to unblock #25188.

@rsimha rsimha merged commit 7316d7c into ampproject:master Oct 22, 2019
@twifkak
Copy link
Copy Markdown
Member

twifkak commented Oct 22, 2019

This was published as https://www.npmjs.com/package/amphtml-validator version 1.0.28.

@dmouse
Copy link
Copy Markdown

dmouse commented Oct 23, 2019

any plans to support node 8.x? Next.js use it vercel/next.js#9172

@rsimha
Copy link
Copy Markdown
Contributor

rsimha commented Oct 29, 2019

any plans to support node 8.x? Next.js use it zeit/next.js#9172

Node 8 support was restored with #25278 and #25279, and should be available with amphtml-validator v1.0.29.

micajuine-ho pushed a commit to micajuine-ho/amphtml that referenced this pull request Dec 27, 2019
@estherkim estherkim deleted the node-10-for-validator branch April 5, 2022 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants