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

fetch branch for bitbucket onprem #242

Merged
merged 1 commit into from
Feb 9, 2023
Merged

fetch branch for bitbucket onprem #242

merged 1 commit into from
Feb 9, 2023

Conversation

devkimittal
Copy link
Collaborator

No description provided.

@devkimittal devkimittal force-pushed the devmittal branch 2 times, most recently from dd80adf to 9a512d7 Compare February 8, 2023 09:03
@@ -0,0 +1,8 @@
{
"id": "refs/heads/mohit/mohit-temp",
Copy link
Contributor

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",
Copy link
Contributor

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",
Copy link
Contributor

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"
},
{
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

@devkimittal
Copy link
Collaborator Author

@rutvijmehta-harness made the changes according to your comments.

@devkimittal devkimittal merged commit 8343d3e into master Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants