Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

include ts.d files in output lib #86

Merged
merged 2 commits into from
May 22, 2023
Merged

Conversation

sreuland
Copy link
Contributor

the *.ts.d files were not being included in the lib output, causing ts validation errors for downstream client usage of types.

Closes #81

@sreuland sreuland requested a review from Shaptic May 20, 2023 00:42
Copy link
Contributor

@Shaptic Shaptic left a comment

Choose a reason for hiding this comment

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

Had a similar realization in stellar/js-stellar-sdk#830, didn't think to check here. Thanks!

package.json Outdated
@@ -27,7 +27,8 @@
"scripts": {
"build": "cross-env NODE_ENV=development yarn _build",
"build:prod": "cross-env NODE_ENV=production yarn _build",
"build:node": "babel --extensions '.ts' --out-dir lib/ src/",
"build:ts": "npx tsc",
Copy link
Contributor

Choose a reason for hiding this comment

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

using yarn run tsc would be better for consistency

@sreuland sreuland merged commit 588d3a0 into stellar:main May 22, 2023
@Shaptic Shaptic mentioned this pull request May 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Could not find a declaration file (TypeScript)
3 participants