-
Notifications
You must be signed in to change notification settings - Fork 15
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
Cannot checkout when using NuKeeper extension on a branch with slash #37
Comments
I think you are right: https://github.com/NuKeeperDotNet/NuKeeper-AzureDevops-extension/blob/master/nukeeper/src/NuKeeper.ts#L18 Could you give me the exact arguments you are using? I can try to replicate and fix the issue later today. |
Yep, thanks. Is bug/31386-xxx the default branch, or do you target that one in the pipeline? For reference: microsoft/azure-pipelines-agent#838 |
I opened up a PR, I think that code change will fix the issue you're experiencing. |
thanks very much, it was the branch that i targeted |
* Update from upstream. (#1) * Pull after branch checkout. Fix for: NuKeeperDotNet/NuKeeper#620 * Updated task version. * Update README.md * docker build * update to use v0.13.0 and set icon * 0.16.0 * use latest always * use these files * delete all bin files * feat(installer): changed to native download and extraction * npm fix packages * version 0.1.65 * fix(Unzip): await unzip (#21) * fix(Unzip): await unzip * fix(Zip): unzip reverted * skip download when already available (#23) * fix(Unzip): await unzip * fix(Zip): unzip reverted * fix(Download): only download when it doesn't exists * fix(git): changed git commands to synchronous (#24) * fix(git): changed git commands to synchronous * fix(version): updated version * update to 0.1.691 using tool lib for download (#25) * update * remove obsolte libs * remove old version of tool-lib * Update README.md * 0.2.2 (#26) * update * remove obsolte libs * remove old version of tool-lib * move to 0.2.2 * update to 0.2.4 * Update explanation about oauth checkmark and update version and use async exectuion * version 0.3.2 * version 0.3.4 * Update README.md * add version capabalities * remove jsonpath-plus * update packages * add group * add installer.ts file * no global * update latest * Make NuKeeper task debuggable * log output to debug console * use slash not dot * remove unneeded variable * Update README.md * Update README.md * release 0.4.7 * move stuff aroudn * moving again * update to latest * Added SourceBranch var to launch.json. * Support slashes in branch. * Corrected spacing.
@thomasmktong Could you try again, new version of extension should be live. |
When running NuKeeper extension on a branch with slash, e.g.
bug/31386-xxx
I found that only the last segment of the branch is sent to NuKeeper, i.e.31386-xxx
withoutbug/
I think somehow the VSTS build variable
Build.SourceBranchName
was used as that one only has last segment of a branch name, ref: https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yamlThe text was updated successfully, but these errors were encountered: