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

ci: stop testing against NodeJS v14, v16 #2446

Merged
merged 8 commits into from
May 22, 2023
Merged

Conversation

wolfy1339
Copy link
Member

@wolfy1339 wolfy1339 commented May 13, 2023

BREAKING CHANGE: Drop support for NodeJS v14, v16

@wolfy1339 wolfy1339 added Type: Breaking change Used to note any change that requires a major version bump Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR labels May 13, 2023
@wolfy1339 wolfy1339 force-pushed the remove-eol-node-versions branch 2 times, most recently from 35ab257 to 475bb2f Compare May 13, 2023 16:27
@wolfy1339 wolfy1339 changed the title ci: stop testing against NodeJS v14 ci: stop testing against NodeJS v14, v16 May 14, 2023
@wolfy1339 wolfy1339 force-pushed the remove-eol-node-versions branch from 5e3fb2e to 5d97da0 Compare May 14, 2023 02:00
@kfcampbell
Copy link
Member

@wolfy1339 Node 16 is still supported until September 2023. What's the justification for stripping it before then?

@wolfy1339
Copy link
Member Author

It was suggested by @gr2m in octokit/webhooks.js#849 (review)

In order to avoid making another breaking release in a couple months

@wolfy1339 wolfy1339 changed the base branch from main to beta May 22, 2023 19:13
@wolfy1339 wolfy1339 enabled auto-merge (squash) May 22, 2023 19:15
@wolfy1339
Copy link
Member Author

Something is up with branch protection. It currently requires node 10 and 12. which were removed a whiles back

@gr2m
Copy link
Contributor

gr2m commented May 22, 2023

Something is up with branch protection. It currently requires node 10 and 12. which were removed a whiles back

most likely we didn't update the beta branch protections, on it

@wolfy1339 wolfy1339 merged commit ba566fc into beta May 22, 2023
@wolfy1339 wolfy1339 deleted the remove-eol-node-versions branch May 22, 2023 19:19
@github-actions
Copy link

🎉 This PR is included in version 3.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @beta Type: Breaking change Used to note any change that requires a major version bump Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants