-
Notifications
You must be signed in to change notification settings - Fork 237
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
fetch branch for bitbucket onprem #242
Conversation
dd80adf
to
9a512d7
Compare
@@ -0,0 +1,8 @@ | |||
{ | |||
"id": "refs/heads/mohit/mohit-temp", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Prefer adding generic names
@@ -3,7 +3,7 @@ | |||
"Namespace": "PRJ", | |||
"Name": "my-repo", | |||
"Perm": null, | |||
"Branch": "master", | |||
"Branch": "notmaster", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: feature_branch
@@ -22,6 +22,7 @@ | |||
"stash.go", | |||
"stash_test.go", | |||
"testdata/branch.json", | |||
"testdata/default_branch.json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you have to add golden as well here?
nit: Alphabetical order
@@ -75,6 +75,10 @@ | |||
"path": "testdata/branch.json", | |||
"kind": "file" | |||
}, | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
9a512d7
to
2a53c8e
Compare
@rutvijmehta-harness made the changes according to your comments. |
2a53c8e
to
df6c516
Compare
No description provided.