-
Notifications
You must be signed in to change notification settings - Fork 179
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
ci(shared-data, components): add github action for publishing packages to npm #14077
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## edge #14077 +/- ##
==========================================
Coverage 70.46% 70.46%
==========================================
Files 2511 1633 -878
Lines 71194 54520 -16674
Branches 8952 4026 -4926
==========================================
- Hits 50165 38420 -11745
+ Misses 18849 15403 -3446
+ Partials 2180 697 -1483
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -61,7 +61,6 @@ setup-js: | |||
yarn | |||
$(MAKE) -C $(APP_SHELL_DIR) setup | |||
$(MAKE) -C $(APP_SHELL_ODD_DIR) setup | |||
$(MAKE) -C $(SHARED_DATA_DIR) setup-js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this call is no longer necessary.
|
||
.PHONY: dist-js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these were purely used for bundling v1 labware defs for old versions of monorepo SPA's
…s to npm In order to support a single source of truth for applications outside of the monorepo (read: "Protocol Library") that rely on fundamental components and geometries within the mono repo, add tag based deploy actions that publish bundled artifacts to NPM.
@@ -115,14 +106,7 @@ export class URLDeck extends React.Component<{}> { | |||
> | |||
{labwareDefV2 ? ( | |||
<LabwareRender definition={labwareDefV2} /> | |||
) : ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this whole directory will never be deployed again. It can be removed when legacy PL is unlisted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We eagerly await the day
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
@@ -115,14 +106,7 @@ export class URLDeck extends React.Component<{}> { | |||
> | |||
{labwareDefV2 ? ( | |||
<LabwareRender definition={labwareDefV2} /> | |||
) : ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We eagerly await the day
echo "No build for ref ${{github.ref}} and event ${{github.event_type}}" | ||
echo 'type=none' >> $GITHUB_OUTPUT | ||
fi | ||
|
||
deploy-components: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deploy-components: | |
deploy-components-storybook: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
…s to npm (#14077) In order to support a single source of truth for applications outside of the monorepo (read: "Protocol Library") that rely on fundamental components and geometries within the mono repo, add tag based deploy actions that publish bundled artifacts to NPM.
Overview
In order to support a single source of truth for applications outside of the monorepo (read:
"Protocol Library") that rely on fundamental components and geometries within the mono repo, add tag
based deploy actions that publish bundled artifacts to NPM.
Changelog
Review requests
Risk assessment