-
Notifications
You must be signed in to change notification settings - Fork 724
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
Implement deployment API support for ant-man and flash previews #960
Conversation
@bitwiseman Tests are implemented and this is ready for review. One change that I wanted feedback on was the addition of a value field to the |
@bitwiseman friendly bump |
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.
Sorry for long delay.
Only one type of change needed. Otherwise looks great.
@bitwiseman Feedback addressed. |
0c6ddae
to
59b61cd
Compare
@@ -8,11 +8,23 @@ | |||
* Indicates that the method/class/etc marked maps to GitHub API in the preview period. | |||
* <p> | |||
* These APIs are subject to change and not a part of the backward compatibility commitment. Always used in conjunction | |||
* with 'deprecated' to raise awareness to clients. | |||
* with 'deprecated' to raise awareness to clients. In addition, it's advised to update the targets documentation to | |||
* signify that the deprecation is required until preview feature being used is promoted to stable. |
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.
👍
Description
Previews
class.GHDeployment
andGHDeploymentStatus
.GHDeploymentBuilder
andGHDeploymentStatusBuilder
.GHDeploymentState
has been updated to supportin_progress
,queued
, andinactive
.GHDeploymentStatus.getTargetUrl()
andGHDeploymentStatusBuilder.targetUrl()
as@Deprecated
.logUrl()
where appropriateGithub Blog Posts For Previews Implemented
Before submitting a PR:
We love getting PRs, but we hate asking people for the same basic changes every time.
master
. Create your PR from that branch.mvn clean compile
locally. This may reformat your code, commit those changes.mvn -D enable-ci clean install site
locally. If this command doesn't succeed, your change will not pass CI.When creating a PR: