Skip to content

fix: add Node.js 10 to CI and loosen engines field#2119

Merged
UziTech merged 5 commits intomasterfrom
node-version-10
Jun 22, 2021
Merged

fix: add Node.js 10 to CI and loosen engines field#2119
UziTech merged 5 commits intomasterfrom
node-version-10

Conversation

@styfle
Copy link
Copy Markdown
Member

@styfle styfle commented Jun 21, 2021

Fixes #2111

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 21, 2021 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/markedjs/markedjs/7ThYuLGW61Koh3MfZ2TjdFTKV9my
✅ Preview: https://markedjs-git-node-version-10-markedjs.vercel.app

@styfle styfle changed the title Add tests for node 10 Add Node.js 10 to CI and loosen engines field Jun 21, 2021
@styfle styfle marked this pull request as ready for review June 21, 2021 23:12
@styfle styfle changed the title Add Node.js 10 to CI and loosen engines field fix: add Node.js 10 to CI and loosen engines field Jun 21, 2021
@styfle styfle requested a review from UziTech June 21, 2021 23:12
@styfle styfle mentioned this pull request Jun 21, 2021
Closed
@UziTech
Copy link
Copy Markdown
Member

UziTech commented Jun 22, 2021 •
edited
Loading

This works but I don't think it solves the problem. V2.0.7 engines field was v8 so this is still dropping node versions without a major version change and it is supporting a node version after eol which is kind of worst of both worlds.

@styfle
Copy link
Copy Markdown
Member Author

styfle commented Jun 22, 2021

@UziTech I just tried Node 8 and it failed so I think we can still say this is a patch version to fix the engines field

@Fleker
Copy link
Copy Markdown

Fleker commented Jun 22, 2021

Thanks for doing this PR.

@calculuschild calculuschild reopened this Jun 22, 2021
@calculuschild
Copy link
Copy Markdown
Contributor

calculuschild commented Jun 22, 2021

Gosh dang. I hate when the close PR button pops up under my thumb on mobile.

@UziTech UziTech merged commit 8659353 into master Jun 22, 2021
@UziTech UziTech deleted the node-version-10 branch June 22, 2021 17:25
github-actions bot pushed a commit that referenced this pull request Jun 22, 2021
@semantic-release-bot
chore(release): 2.1.2 [skip ci]
c7b4664 
## [2.1.2](v2.1.1...v2.1.2) (2021-06-22)

### Bug Fixes

* add Node.js 10 to CI and loosen `engines` field ([#2119](#2119)) ([8659353](8659353))
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 22, 2021

🎉 This PR is included in version 2.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Logiclayer1111 pushed a commit to Logiclayer1111/marked that referenced this pull request Apr 20, 2026
@semantic-release-bot
chore(release): 2.1.2 [skip ci]
dcb6189 
## [2.1.2](markedjs/marked@v2.1.1...v2.1.2) (2021-06-22)

### Bug Fixes

* add Node.js 10 to CI and loosen `engines` field ([#2119](markedjs/marked#2119)) ([3804dca](markedjs/marked@3804dca))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@UziTech UziTech UziTech approved these changes

@calculuschild calculuschild calculuschild approved these changes

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Breaking Change in 2.1.1

4 participants

@styfle @UziTech @Fleker @calculuschild