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

chore: drop Node 10 support #623

Merged
merged 1 commit into from
May 18, 2021
Merged

chore: drop Node 10 support #623

merged 1 commit into from
May 18, 2021

Conversation

mpeyper
Copy link
Member

@mpeyper mpeyper commented May 18, 2021

What:

Drop node 10 support

Why:

Fixes #621

How:

Remove node 10.13 and 10 from node version matrix in github actions.

Checklist:

  • Ready to be merged

@mpeyper mpeyper requested a review from MichaelDeBoey May 18, 2021 11:14
@codecov
Copy link

codecov bot commented May 18, 2021

Codecov Report

Merging #623 (0d85f6c) into main (882bcfe) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #623   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          218       218           
  Branches        26        26           
=========================================
  Hits           218       218           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 882bcfe...0d85f6c. Read the comment docs.

@mpeyper mpeyper requested a review from joshuaellis May 18, 2021 11:17
@MichaelDeBoey MichaelDeBoey added the BREAKING CHANGE This change will require a major version bump label May 18, 2021
@MichaelDeBoey MichaelDeBoey changed the title Drop node 10 support chore: drop Node 10 support May 18, 2021
@mpeyper mpeyper force-pushed the feat/drop-node-10 branch from 24aaca3 to da3dedb Compare May 18, 2021 12:25
package.json Show resolved Hide resolved
@mpeyper
Copy link
Member Author

mpeyper commented May 18, 2021

Can somebody just rereview this and check the commit message is correct for a major bump. I always seem to mess up the order of the breaking changes, fixes and co-authors an either the wrong version gets published or the release notes get all messed up.

@joshuaellis
Copy link
Member

It should cause a breaking change release because it's in the body. I couldn't say about release notes, they're still a mystery to me.

@mpeyper
Copy link
Member Author

mpeyper commented May 18, 2021

I found this which was linked from the semantic-release docs which suggests I need 2 blank lines after the breaking change summary otherwise it will think it is the breaking change description... I'll put an extra one in to be safe 🤷‍♂️

BREAKING CHANGE: Node 10 is no longer supported

Fixes #621

Co-authored-by: Michaël De Boey <[email protected]>
@mpeyper mpeyper force-pushed the feat/drop-node-10 branch from da3dedb to 0d85f6c Compare May 18, 2021 12:50
Copy link
Member

@joshuaellis joshuaellis left a comment

Choose a reason for hiding this comment

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

Yeah the breaking change system always gets me :(

@mpeyper
Copy link
Member Author

mpeyper commented May 18, 2021

Annoyingly, git is trimming the double blank line into a single new line when I commit... oh well, guess I'll just have to double check the release notes after it runs.

@mpeyper mpeyper merged commit 39dd228 into main May 18, 2021
@joshuaellis
Copy link
Member

I didn't double line break yesterday and it worked for me if incl release notes...

@github-actions
Copy link

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mpeyper mpeyper deleted the feat/drop-node-10 branch May 18, 2021 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BREAKING CHANGE This change will require a major version bump released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop Node 10 support
3 participants