-
Notifications
You must be signed in to change notification settings - Fork 56
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
Project deploy changes #561
Conversation
packages/cli/lib/projects.js
Outdated
@@ -232,11 +232,102 @@ const pollBuildStatus = async (accountId, name, buildId) => { | |||
}); | |||
}; | |||
|
|||
const pollDeployStatus = async (accountId, name, deployId) => { |
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 is very similar to the build status polling logic. Will definitely DRY this up- time permitting, I'll do that in this PR
I haven't dug into this yet, but I wanted to mention that IMO it's OK to get this deploy work out without handling the " Update hs project upload to be aware of deployments" work. If you want you could handle that in a separate PR. |
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.
Code looks good!
Description and Context
hs project deploy
for new deploy serviceScreenshots
TODO
Who to Notify