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

using tag like commit message does not allow to make request #88

Open
kalabiyau opened this issue Mar 4, 2014 · 4 comments
Open

using tag like commit message does not allow to make request #88

kalabiyau opened this issue Mar 4, 2014 · 4 comments

Comments

@kalabiyau
Copy link
Collaborator

given last commit in HEAD looks like [TAGNAME] done stuff

having latter will not allow to make a review request.

@xystushi
Copy link
Collaborator

xystushi commented Mar 7, 2014

Could you please explain how to reproduce the problem?

@kalabiyau
Copy link
Collaborator Author

given you have a commit in your branch with square brackets []
like [TAG] this is commit message

if you will try to git review create it will break execution

╭─kalabiyau@kalabiyau  ~/Code ‹ruby-2.0.0› ‹review_140321_caramba*› 
╰─$ git ci -am "[tag] commit"
[review_140321_caramba 8311faf] [tag] commit
 1 file changed, 2 insertions(+), 1 deletion(-)
╭─kalabiyau@kalabiyau  ~/Code ‹ruby-2.0.0› ‹review_140321_caramba› 
╰─$ git review create
Faraday::Builder is now Faraday::RackBuilder.
fatal: 'tag' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
error: Execution of git-review stopped.

xystushi added a commit to xystushi/git-review that referenced this issue Mar 22, 2014
@xystushi
Copy link
Collaborator

Yes. I can confirm this bug.

It is because currently in remote_for_branch() we are trying to match remote tracking branch like [origin/master]. If there's no remote branch and the commit message happens to start with [some tag], it will mistakenly match some tag to remote branch name.

This is fixed in #90.

b4mboo pushed a commit that referenced this issue Mar 28, 2014
Fix #88, allow [tag] in commit message.
@b4mboo
Copy link
Owner

b4mboo commented Mar 28, 2014

Fix is in development branch. I plan to release a new version before GSoC anyways (and thus merge development into master). Gimme a couple of days to regain a feeling about what's in the development branch right now and what can be merged or cherry-picked.

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

3 participants