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

gemspec commits to master branch instead of current branch #194

Open
treeder opened this issue Jul 1, 2011 · 10 comments
Open

gemspec commits to master branch instead of current branch #194

treeder opened this issue Jul 1, 2011 · 10 comments
Labels

Comments

@treeder
Copy link

treeder commented Jul 1, 2011

No description provided.

@technicalpickles
Copy link
Owner

Sure, that seems fine. Just curious, but what branch do you develop/release from?

@treeder
Copy link
Author

treeder commented Jul 2, 2011

Usually master, but we have a beta branch that we're releasing beta's from. Also, the same issue would occur if you were doing bug fixes on a previous release.

@sarahmei
Copy link

+1. We have an internal gem that we are maintaining for 1.8.6 and 1.8.7 on separate branches.

@mxgrn
Copy link

mxgrn commented Aug 16, 2011

+1.
I have a couple of extra branches where development of previous versions is taking place. Thus, after releasing the version, say, 0.7.1, that would be in 'master', I may need to switch to the '0-6' branch and release the version, say, 0.6.5.

(In short, a story similar to sarahmei's)

@technicalpickles
Copy link
Owner

I'm cool with this as a feature. We've had to do a similar thing when doing preleases of a gem.

I haven't had a ton of time to make it happen though. If someone were to send a pull request before my next jeweler hack session, I'd be happy to review and see about getting it merged.

@grosser
Copy link

grosser commented Sep 23, 2011

#207

@treeder
Copy link
Author

treeder commented Sep 28, 2011

@technicalpickles could you please merge that pull request to fix this?

@technicalpickles
Copy link
Owner

I've been pretty swamped recently, so haven't had much time for some TLC of jeweler. I will get it out as soon as I can.

A quick pass of the pull request looks good though.

On Sep 28, 2011, at 6:15 PM, Travis Reeder wrote:

@technicalpickles could you please merge that pull request to fix this?

Reply to this email directly or view it on GitHub:
#194 (comment)

@treeder
Copy link
Author

treeder commented Oct 12, 2011

Any word on this?

@railsformers
Copy link

We need this feature too. Some progress in merging pull request?

@flajann2 flajann2 added the bug label Oct 22, 2016
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

6 participants