Skip to content
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

phing release creates public GitHub Pro release #157

Open
raamdev opened this issue Feb 21, 2017 · 2 comments
Open

phing release creates public GitHub Pro release #157

raamdev opened this issue Feb 21, 2017 · 2 comments
Labels

Comments

@raamdev
Copy link
Contributor

raamdev commented Feb 21, 2017

While publishing s2Member Pro v170221 today (using Phings v170105.65238), I noticed that it published a public GitHub release with the zip file attached; it shouldn't be doing that for the Pro version. The Pro GitHub release should be saved as a Draft.

2017-02-20_19-46-52

I just used Phings v170105.65238 to publish a Comet Cache release and the Pro GitHub release was not published as public... so I have no idea why it got published as public for s2Member Pro.

@raamdev raamdev added the bug label Feb 21, 2017
@raamdev raamdev changed the title phing release creates public Pro release phing release creates public GitHub Pro release Feb 21, 2017
@jaswrks
Copy link
Contributor

jaswrks commented Mar 9, 2017

This happens because unfortunately, s2Member doesn't have a namespace that the Phing system can use to detect the pro status. In the eyes of our Phing system, s2Member Pro is not a pro version of our software like so many of our other packages are.

I'm not seeing an easy way around this at the moment, mostly because forcing it into a pro status in the eyes of our Phing system comes with some other things I'll need to review carefully.

For now, the easiest way around this problem is to add the draft flag.

$ phing release -D draft=true

@raamdev
Copy link
Contributor Author

raamdev commented Mar 10, 2017

@jaswrks Cool, that will suffice for me. Thanks! I'm going to keep track of that modified step in this new s2Member Release Process template that I'm working on: https://github.com/websharks/s2member.com/wiki/Release-Process-Template

Feel free to close this GitHub issue if you don't see any further reason to keep it around.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants