Skip to content

Conversation

@martinvanzijl
Copy link
Contributor

Description

Allows having the "refs/" prefix in parameter to GHRepository.getRef(). This is optional, so "refs/heads/master" will work, as well as "heads/master".

Fixes #518.

@martinvanzijl
Copy link
Contributor Author

There is a CI failure about "addCollaborators". From the log:

[ERROR] addCollaborators(org.kohsuke.github.GHRepositoryTest) Time elapsed: 0.046 s <<< ERROR!

I don't think I've changed this method, though.

@@ -1,25 +0,0 @@
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@martinvanzijl It looks like you modified the test resources for this, whether you meant to or not. This would be where the error is coming from.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bitwiseman Sorry, my mistake. Should be fixed now.

@bitwiseman bitwiseman merged commit 95f83d1 into hub4j:master Apr 6, 2020
@martinvanzijl martinvanzijl deleted the issue_518_allow_getref_with_prefix branch April 6, 2020 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistency between createRef and getRef

2 participants