Skip to content

Conversation

@darrelmiller
Copy link
Member

No description provided.

@darrelmiller
Copy link
Member Author

@PerthCharern Before merging this, I'd like to try disabling the auto deploy from AppVeyor and see if the VSTS deploy of the signed package works.

@PerthCharern
Copy link
Contributor

No problem Darrel. Would you like me to turn that off, or will you do that yourself?

@PerthCharern
Copy link
Contributor

If you are going to do it, I suggest we turn off the auto-deploy in GitHub (so that the push doesn't trigger appveyor) rather than turning off the deployment in AppVeyor. It's much easier to re-enable it later.

Just uncheck the "Active" box here: https://github.com/Microsoft/OpenAPI.NET/settings/hooks/18999895

@darrelmiller
Copy link
Member Author

@PerthCharern I have turned off the AppVeyor hooks. If you can approve this PR, I will merge and attempt to release from VSTS

@darrelmiller
Copy link
Member Author

@PerthCharern @xuzhg Could someone approve this for me? Please :-)

@darrelmiller darrelmiller merged commit c83ae29 into master May 2, 2018
@darrelmiller darrelmiller deleted the dm/release-1.0.0-beta017 branch May 2, 2018 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants