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

fix(build): bump @octokit/core to 5.x #2543

Merged
merged 5 commits into from
Sep 19, 2023
Merged

fix(build): bump @octokit/core to 5.x #2543

merged 5 commits into from
Sep 19, 2023

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Jul 19, 2023

Description

BREAKING CHANGE: @octokit/core 5.0.0 drops support for Node 14 and 16.

Testing shows that at least Node 16 still works if you import node-fetch, so technically, if your setup does not enforce Node version, you can still use this package.

Test plan

cd incubator/build
yarn build --dependencies
yarn rnx-build -p ios --device-type simulator ../../packages/test-app

@tido64 tido64 mentioned this pull request Aug 7, 2023
1 task
@tido64 tido64 force-pushed the tido/build/node-18-only branch from b00f801 to 6ae007b Compare September 15, 2023 06:25
BREAKING CHANGE: `@octokit/core` 5.0.0 drops support for Node 14 and 16.

Testing shows that at least Node 16 still works if you import
`node-fetch`, so technically, if your setup does not enforce Node
version, you can still use this package.
@tido64 tido64 force-pushed the tido/build/node-18-only branch from 6ae007b to b297d96 Compare September 18, 2023 06:50
@tido64 tido64 marked this pull request as ready for review September 18, 2023 06:50
Copy link
Contributor

@kelset kelset left a comment

Choose a reason for hiding this comment

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

it's working correctly in my test 👍

@tido64 tido64 merged commit 9c863db into main Sep 19, 2023
17 checks passed
@tido64 tido64 deleted the tido/build/node-18-only branch September 19, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants