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-rc uploads generic zip filename to GitHub Release #112

Open
raamdev opened this issue Jun 11, 2016 · 8 comments
Open

phing release-rc uploads generic zip filename to GitHub Release #112

raamdev opened this issue Jun 11, 2016 · 8 comments

Comments

@raamdev
Copy link
Contributor

raamdev commented Jun 11, 2016

The filename that gets included on the GitHub Release by the phing release-rc command is a generic comment-mail.zip filename, as opposed to something that includes the version number (e.g., comment-mail-v160611-RC.zip):

2016-06-10_22-15-57

Including the version number in the filename would be very helpful to avoid confusion (when somebody downloads the zip file, they'll be able to see what version of Comment Mail is included in that zip file).

@jaswrks
Copy link
Contributor

jaswrks commented Jun 18, 2016

Noting that the next release of the hub tool will make this easier so I'm going to wait for a bit and see if that is made available soon. Referencing: https://github.com/github/hub/blob/master/commands/release.go#L63

@jaswrks
Copy link
Contributor

jaswrks commented Aug 24, 2016

Noting that 'hub' v2.2.5 is now available and it includes the changes we need to implement this properly. However, the Homebrew package has not been updated just yet, so still waiting on that.

@jaswrks
Copy link
Contributor

jaswrks commented Sep 8, 2016

Noting that the Homebrew package has been updated. This is now 'ready for work'.

All Team Members Should Run

$ brew update;
$ brew upgrade hub;

@jaswrks
Copy link
Contributor

jaswrks commented Dec 21, 2016

Noting that I attempted to implement this, but as of Hub v2.2.9 this does not appear to work as documented. Leaving this as-is for now.

@raamdev
Copy link
Contributor Author

raamdev commented Dec 21, 2016

@jaswsinc Noting that you still have a changelog entry in https://github.com/websharks/phings/blob/dev/CHANGELOG.md that indicates we're using named uploads, even though that change has been reversed.

@jaswrks
Copy link
Contributor

jaswrks commented Dec 21, 2016

Thank you for catching that.

@jaswrks
Copy link
Contributor

jaswrks commented Dec 21, 2016

A maintainer tells me this is currently available only in the master branch.
Which is their dev branch: https://github.com/github/hub

@jaswrks
Copy link
Contributor

jaswrks commented Dec 21, 2016

Flagging this as blocked. Awaiting next major release of Hub. See previous comment.

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

No branches or pull requests

2 participants