Skip to content

Conversation

asedge
Copy link
Contributor

@asedge asedge commented Jul 10, 2014

Feedback is always appreciated!

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.11%) when pulling 5bf1363 on asedge:annotated_tags into f959bd4 on gitlabhq:master.

@asedge
Copy link
Contributor Author

asedge commented Jul 10, 2014

This is strange. The test passes on my laptop and my workstation (Ruby 1.9.3 & Git 2.0). I'm also able to use "git rev-parse tag_name^{}" on my production gitlab server using "/opt/gitlab/embedded/bin/git" (Git 1.8.5.3). I'll look at this again later tonight.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.11%) when pulling c0c082f on asedge:annotated_tags into f959bd4 on gitlabhq:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling df90f59 on asedge:annotated_tags into f959bd4 on gitlabhq:master.

@asedge
Copy link
Contributor Author

asedge commented Jul 12, 2014

OK, CI build passed. Just waiting for feedback now. Thanks!

@jvanbaarsen
Copy link
Contributor

@asedge Thanks
@randx Looks ok to merge

@asedge
Copy link
Contributor Author

asedge commented Jul 13, 2014

@jvanbaarsen No problem! I already put in most of the work to allow you to create annotated tags from the UI too: https://github.com/asedge/gitlabhq/commits/annotated_tags. That PR should be filed soon.

dzaporozhets added a commit that referenced this pull request Jul 15, 2014
Adding support+tests for annotated tags.
@dzaporozhets dzaporozhets merged commit 59bffe6 into gitlabhq:master Jul 15, 2014
@asedge asedge deleted the annotated_tags branch July 17, 2014 00:01
maxlazio pushed a commit that referenced this pull request Jun 5, 2019
Go implementation for git-upload-archive

Closes #162

See merge request gitlab-org/gitlab-shell!307
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants