Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(gatsby): Fixes incorrect type #31358

Merged
merged 1 commit into from
May 11, 2021
Merged

fix(gatsby): Fixes incorrect type #31358

merged 1 commit into from
May 11, 2021

Conversation

LekoArts
Copy link
Contributor

Description

Follow-up of #30819

The error wasn't caught because of: #31324 (comment)

Related Issues

Fixes #31324

@LekoArts LekoArts added the topic: TypeScript Issues and PRs related to TS in general, public typings or gatsby-plugin-typescript label May 11, 2021
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 11, 2021
@LekoArts LekoArts removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 11, 2021
@@ -251,7 +251,8 @@
"prebuild": "rimraf dist && rimraf cache-dir/commonjs",
"postinstall": "node scripts/postinstall.js",
"prepare": "cross-env NODE_ENV=production npm run build",
"watch": "rimraf dist && mkdir dist && npm run build:internal-plugins && npm run build:rawfiles && npm run build:src -- --watch"
"watch": "rimraf dist && mkdir dist && npm run build:internal-plugins && npm run build:rawfiles && npm run build:src -- --watch",
"typecheck": "tsc --noEmit"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For easier checking with a modified tsconfig

@@ -12,7 +12,6 @@
"strictFunctionTypes": true,
"strictPropertyInitialization": true,
"noFallthroughCasesInSwitch": true,
"skipLibCheck": true,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Duplicate entry

@LekoArts LekoArts merged commit f629806 into master May 11, 2021
@LekoArts LekoArts deleted the fix-31324 branch May 11, 2021 09:37
vladar pushed a commit that referenced this pull request May 11, 2021
vladar pushed a commit that referenced this pull request May 11, 2021
(cherry picked from commit f629806)

Co-authored-by: Lennart <lekoarts@gmail.com>
axe312ger pushed a commit that referenced this pull request May 20, 2021
This was referenced Mar 14, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
topic: TypeScript Issues and PRs related to TS in general, public typings or gatsby-plugin-typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Type error in main gatsby package index.d.ts: Type 'Node' is not generic
2 participants