Merged in chore/semver-build-info (pull request #3418)
build info should have + sign in semver * build info should have + sign in semver
This commit is contained in:
@@ -30,19 +30,19 @@ describe("getSemver", () => {
|
||||
|
||||
it("should append cleaned branch name for non-named branches", () => {
|
||||
expect(getSemver("package", "1.0.0", "feature/my-feature")).toBe(
|
||||
"package@1.0.0-feature-my-feature"
|
||||
"package@1.0.0+feature-my-feature"
|
||||
)
|
||||
})
|
||||
|
||||
it("should handle special characters in non-named branches", () => {
|
||||
expect(getSemver("package", "1.0.0", "fix/bug#123")).toBe(
|
||||
"package@1.0.0-fix-bug-123"
|
||||
"package@1.0.0+fix-bug-123"
|
||||
)
|
||||
})
|
||||
|
||||
it("should lowercase and trim non-named branches", () => {
|
||||
expect(getSemver("package", "1.0.0", " MY-BRANCH ")).toBe(
|
||||
"package@1.0.0-my-branch"
|
||||
"package@1.0.0+my-branch"
|
||||
)
|
||||
})
|
||||
|
||||
@@ -53,6 +53,6 @@ describe("getSemver", () => {
|
||||
"1.0.0",
|
||||
"feature/very-long-branch-name-that-exceeds-limit"
|
||||
)
|
||||
).toBe("package@1.0.0-feature-very-long-br")
|
||||
).toBe("package@1.0.0+feature-very-long-br")
|
||||
})
|
||||
})
|
||||
|
||||
@@ -26,7 +26,7 @@ export function getSemver(
|
||||
.replace(/[^a-zA-Z0-9]/g, "-")
|
||||
.toLowerCase()
|
||||
|
||||
return `${packageName}@${version}-${cleanedBranch}`
|
||||
return `${packageName}@${version}+${cleanedBranch}`
|
||||
}
|
||||
|
||||
return `${packageName}@${version}`
|
||||
|
||||
Reference in New Issue
Block a user