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

Update dependencies 2021-06-26 #156

Closed
wants to merge 10 commits into from
Closed

Conversation

Simran-B
Copy link

@Simran-B Simran-B commented Jun 27, 2021

Update dependencies to latest except husky (v5 moves to a somewhat messy file-based approach, but let me know if you want to upgrade anyway).

Fixes #133 and #155.

Breaking changes:

REST endpoint methods are now exposed under a nested object rest by Octokit. For github-script, this means that endpoints are now accessed via github.rest.{endpoint} rather than github.{endpoint}. Also see:

jest 27 removes support for Node.js 11 and 13 (now only supporting LTS versions and Node.js 15). You may want to up the Node.js version in this workflow:

with: {node-version: 13.x}

This changes the definition of the github object! The endpoints are now accessed via github.rest.{endpoint} rather than `github.{endpoint}`.
Removes Node.js 11 and 13 support
* @typescript-eslint/eslint-plugin to v4.28.0
* @typescript-eslint/parser to v4.28.0
* @vercel/ncc to v0.28.6
* eslint to v7.29.0
* eslint-config-prettier to v8.3.0
* husky to v4.3.8
* prettier to v2.3.2
* typescript to v4.3.4
@Simran-B Simran-B requested a review from a team as a code owner June 27, 2021 00:07
@joshmgross
Copy link
Member

jest 27 removes support for Node.js 11 and 13 (now only supporting LTS versions and Node.js 15). You may want to up the Node.js version in this workflow

Thanks! Since javascript actions are run on Node 12 LTS, we should be using that in the workflow rather than 13.

@joshmgross joshmgross added the dependencies Pull requests that update a dependency file label Jun 28, 2021
@joshmgross
Copy link
Member

I updated the workflows in #159 to resolve that issue with Jest.

It looks like the Licensed check is failing, let me know if you need help updating that. https://github.com/actions/github-script/pull/156/checks?check_run_id=2935017298

@Simran-B
Copy link
Author

@joshmgross The licenses should be sorted out.

I noticed that the pre-commit hook runs under Linux, but I believe it didn't under Windows. Maybe a reason to upgrade husky after all, but that can be a separate PR and I also need to test this again in a clean Windows environment.

@github-actions
Copy link

github-actions bot commented Sep 4, 2021

This pull request is stale because it has been open for 60 days with no activity. Remove the "Stale" label or comment on the pull request, or it will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 4, 2021
@github-actions
Copy link

This pull request has been marked as stale and closed due to no activity on it.

@github-actions github-actions bot closed this Sep 11, 2021
@Timmmy-nlb
Copy link

Sorry, but is there something I can do to help on this PR to solve issue #155?

@Simran-B
Copy link
Author

The maintainers would only need to merge the PR, then tag and release the changes as v5 (semver-major as it contains breaking changes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: github.repos.createOrUpdateEnvironment is not a function - Octokit.js Support up-to-date?
3 participants