Skip to content

Refactor: Bring everything up to speed with new deps and node 18 #79

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

Merged
merged 3 commits into from
Mar 21, 2025

Conversation

jpmcb
Copy link
Owner

@jpmcb jpmcb commented Aug 20, 2023

* Refactor: Bring everything up to speed with new deps and node 20

* Based on works of John McBride (@jpmcb)
* Update dependencies:
  + Bump:
    + @typescript-eslint/eslint-plugin: ^6.7.0
    + @typescript-eslint/parser: ^6.7.0
    + eslint: ^8.49.0
    + typescript: ^5.2.2
  + Add:
    + eslint-import-resolver-typescript: ^3.6.0
      Required for eslint to work with typescript import resolver
    + msw: ^1.3.1
      Introduced to replace nock
  + Replace deprecated:
    + Replace @zeit/ncc with @vercel/ncc, upstream suggestion
      Build error: digital envelope routines::unsupported
  + Drop:
    + nock due to the issue [1] related to the fetch experimental
      feature of nodejs
* Refactor code:
  + TypeScript compile errors
  + ESLint errors and warnings
  + Tests reimplement GitHub API responses mockup using msw/node instead
    of nock
* Declare the action runs using node20

---
[1] nock/nock#2397

Signed-off-by: Neutron Soutmun <[email protected]>

* chore: cosmetic apply with prettier

Signed-off-by: Neutron Soutmun <[email protected]>

---------

Signed-off-by: Neutron Soutmun <[email protected]>
@jpmcb jpmcb marked this pull request as ready for review March 21, 2025 19:30
@jpmcb jpmcb merged commit a55b50d into main Mar 21, 2025
3 of 4 checks passed
@jpmcb jpmcb deleted the dep-bumps branch March 21, 2025 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants