Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

bexnoss
Copy link

@bexnoss bexnoss commented Jan 31, 2022

Description:
This PR enables the generation of TypeScript declaration files.

Related issue:
Improves the workaround from #38 (comment).

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Copy link

@pzhlkj6612 pzhlkj6612 left a comment

Choose a reason for hiding this comment

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

One more step: make git track the generated ".d.ts" and ".d.ts.map" files.

@bexnoss
Copy link
Author

bexnoss commented Jan 31, 2022

I didn't notice that dist/ is tracked. This also simplifies the workaround in #38. Thanks for the heads up!

@bexnoss
Copy link
Author

bexnoss commented Jan 31, 2022

This is blocked by the usage of ncc for building. See #306

@dmitry-shibanov
Copy link
Contributor

Hello @bexnoss. Thank you for your pull request. Would the initial issue be resolved by changing the build command to: tsc && ncc build -o dist/setup src/setup-python.ts && ncc build -o dist/cache-save src/cache-save.ts?

@bexnoss
Copy link
Author

bexnoss commented Feb 1, 2022

Hello @bexnoss. Thank you for your pull request. Would the initial issue be resolved by changing the build command to: tsc && ncc build -o dist/setup src/setup-python.ts && ncc build -o dist/cache-save src/cache-save.ts?

Hello, yes running tsc is what's missing for the workaround from #38 to work.

@bexnoss
Copy link
Author

bexnoss commented Feb 1, 2022

Superseded by #329

@bexnoss bexnoss closed this Feb 1, 2022
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.

3 participants