-
Notifications
You must be signed in to change notification settings - Fork 0
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: change to nx release #255
base: main
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for pine-design-system failed.
|
45c2ed6
to
d38d404
Compare
4e47a6e
to
ab2d240
Compare
ab2d240
to
ae69b9d
Compare
ae69b9d
to
384ec3f
Compare
name: "Pine Continuous Deployment" | ||
|
||
on: | ||
## We will uncomment this when we settle on a deployment schedule |
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.
Do we want to set this schedule prior to merging?
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.
If that's what we want. This could also be run manually like it is currently today.
.prettierignore
Outdated
/dist | ||
/coverage | ||
/.nx/cache | ||
/.nx/workspace-data |
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.
Minor note, eol missing
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.
Minor comments, otherwise looks good!
Description
This is a re-do for PR #232