Skip to content

fix TS2322 and update @typescript-eslint/*#479

Merged
yoshinorin merged 8 commits intohexojs:masterfrom
dimaslanjaka:master
May 7, 2023
Merged

fix TS2322 and update @typescript-eslint/*#479
yoshinorin merged 8 commits intohexojs:masterfrom
dimaslanjaka:master

Conversation

@dimaslanjaka
Copy link
Contributor

@dimaslanjaka dimaslanjaka commented May 6, 2023

fix(ts): error TS2322

Type 'string | void | Buffer' is not assignable to type 'string'.

Type 'void' is not assignable to type 'string'.

succesful build on node 14.

dimaslanjaka@22faf4f

chore(deps-dev): update typescript-eslint/*

update dev dependencies as below list:

  • @typescript-eslint/eslint-plugin@5.59.2
  • @typescript-eslint/parser@5.59.2

Type 'string | void | Buffer' is not assignable to type 'string'.

Type 'void' is not assignable to type 'string'.
- @typescript-eslint/eslint-plugin@5.59.2
- @typescript-eslint/parser@5.59.2
@dimaslanjaka dimaslanjaka changed the title improve some fixes fix TS2322 and update @typescript-eslint/* May 6, 2023
@coveralls
Copy link

coveralls commented May 6, 2023

Coverage Status

Coverage: 84.151% (-0.06%) from 84.211% when pulling c472981 on dimaslanjaka:master into 40cdf2a on hexojs:master.

Copy link
Member

@SukkaW SukkaW left a comment

Choose a reason for hiding this comment

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

It seems that you are using an outdated version of npm. Please upgrade to at least npm@8 and re-generate the package-lock.json

@dimaslanjaka
Copy link
Contributor Author

It seems that you are using an outdated version of npm. Please upgrade to at least npm@8 and re-generate the package-lock.json

reinstalled with npm@8 dimaslanjaka@d4f25a7

@dimaslanjaka dimaslanjaka requested a review from SukkaW May 6, 2023 07:39
@yoshinorin yoshinorin merged commit ee1535a into hexojs:master May 7, 2023
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.

4 participants