fix(ci): build-ts before push to npm #14357
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
There is no typing present in 0.1.1 of
Test Plan
If this is the correct fix, merge it then tag - or - tag this branch? to test a new deploy.
Changelog
make build-ts
in the GitHub Actions job that publishes forshared-data
make build-ts
in the GitHub Actions job that publishes forcomponents
Review requests
opentrons/components/webpack.config.js
Line 6 in 4371e79
barrel.ts
butopentrons/components/package.json
Line 6 in 4371e79
Risk assessment
None to any functional code. CI only change to build types before distributing components and shared-data.