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

Typescript builds fail on version 36.2.1 #2084

Closed
3 tasks done
Tyresius92 opened this issue Feb 5, 2022 · 5 comments
Closed
3 tasks done

Typescript builds fail on version 36.2.1 #2084

Tyresius92 opened this issue Feb 5, 2022 · 5 comments
Labels
Type: Bug 🐛 Oh no! A bug or unintentional behavior Type: TypeScript Issues related to typescript or type definitions

Comments

@Tyresius92
Copy link

Tyresius92 commented Feb 5, 2022

Bugs and Questions

Checklist

  • I have read through the FAQ and Guides before asking a question

  • I am using the latest version of Victory

  • I've searched open issues to make sure I'm not opening a duplicate issue

The Problem

Our TypeScript build is failing when upgrading from 36.2.0 to 36.2.1 due to missing types. We believe the issue was introduced in #2058

The error message output is:

> tsc --project ./tsconfig.build.json --outDir ./dist

node_modules/victory/lib/index.d.ts:114:5 - error TS2305: Module '"victory-selection-container"' has no exported member 'SelectionHelpers'.

114     SelectionHelpers,
        ~~~~~~~~~~~~~~~~

node_modules/victory/lib/index.d.ts:115:5 - error TS2305: Module '"victory-selection-container"' has no exported member 'selectionContainerMixin'.

115     selectionContainerMixin
        ~~~~~~~~~~~~~~~~~~~~~~~


Found 2 errors.

Reproduction

I've created a minimal reproduction here: https://github.com/Tyresius92/victory-bug-reproduction

Alternatively, you can create a reproduction of the issue by:

  1. Create a new React application with npx create-react-app my-app --template typescript
  2. Update skipLibCheck to false in tsconfig.json
  3. Update the build command in package.json to tsc --outDir ./dist
  4. Run npm run build
@becca-bailey
Copy link
Contributor

Thanks for letting me know! This is probably related to #2058.

@becca-bailey becca-bailey added Type: Bug 🐛 Oh no! A bug or unintentional behavior Type: TypeScript Issues related to typescript or type definitions labels Feb 8, 2022
@kaiyoma
Copy link

kaiyoma commented Feb 8, 2022

Same problem here. Type checking fails in my project now after upgrading to 36.2.1.

@becca-bailey
Copy link
Contributor

I'm looking into this today. There might be a quick fix, otherwise I will just revert 2058.

@jasherman
Copy link

@becca-bailey Any update on resolving this?

@becca-bailey
Copy link
Contributor

#2058 Has been reverted, and I just published v36.2.2. Pulling the latest version should resolve this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug 🐛 Oh no! A bug or unintentional behavior Type: TypeScript Issues related to typescript or type definitions
Projects
None yet
Development

No branches or pull requests

4 participants