Skip to content

Commit 5e899cb

Browse files
authored
re-enable release proposal dont-land-on checks (#5665)
1 parent 3e3d7d6 commit 5e899cb

File tree

2 files changed

+5
-20
lines changed

2 files changed

+5
-20
lines changed

scripts/release/proposal.js

+1-12
Original file line numberDiff line numberDiff line change
@@ -66,21 +66,10 @@ try {
6666

6767
pass(`v${releaseLine}.x`)
6868

69-
const diffCmd =
70-
`branch-diff --user DataDog --repo dd-trace-js --exclude-label=semver-major,dont-land-on-v${releaseLine}.x`
69+
const diffCmd = 'branch-diff --user DataDog --repo dd-trace-js --exclude-label=semver-major'
7170

7271
start('Determine version increment')
7372

74-
const legacyDiff = capture(`${diffCmd} --require-label=dont-land-on-v${releaseLine}.x v${releaseLine}.x ${main}`)
75-
76-
if (legacyDiff) {
77-
// TODO: Re-enable this when the offending PR commits have landed properly.
78-
// fatal(
79-
// `The "dont-land-on-v${releaseLine}.x" label is no longer supported.`,
80-
// 'Please remove the label from any offending PR to continue.'
81-
// )
82-
}
83-
8473
const { DD_MAJOR, DD_MINOR, DD_PATCH } = require('../../version')
8574
const lineDiff = capture(`${diffCmd} --markdown=true v${releaseLine}.x ${main}`)
8675

scripts/release/validate.js

+4-8
Original file line numberDiff line numberDiff line change
@@ -37,15 +37,16 @@ try {
3737

3838
start('Pull release branch')
3939

40-
const proposalBranch = params[0] || capture('git rev-parse --abbrev-ref HEAD')
40+
const currentBranch = capture('git rev-parse --abbrev-ref HEAD')
41+
const proposalBranch = params[0] || currentBranch
4142
const tempBranch = randomUUID()
4243
const newVersion = proposalBranch.match(/^v([0-9]+\.[0-9]+\.[0-9]+).+/)[1]
4344
const releaseLine = newVersion.match(/^([0-9]+).+/)[1]
4445

4546
// Restore current branch on success.
4647
process.once('exit', code => {
4748
if (code === 0) {
48-
run(`git checkout ${proposalBranch}`)
49+
run(`git checkout ${currentBranch}`)
4950
}
5051

5152
run(`git branch -D ${tempBranch}`)
@@ -57,12 +58,7 @@ try {
5758

5859
pass()
5960

60-
const diffCmd = [
61-
'branch-diff',
62-
'--user DataDog',
63-
'--repo dd-trace-js',
64-
`--exclude-label=semver-major,dont-land-on-v${releaseLine}.x`
65-
].join(' ')
61+
const diffCmd = 'branch-diff --user DataDog --repo dd-trace-js --exclude-label=semver-major'
6662

6763
start('Validate differences between proposal and main branch.')
6864

0 commit comments

Comments
 (0)