Skip to content

Commit 3ddc20a

Browse files
authored
[release-old] use backport NPM dist tag instead of stable for backports (#79612)
PR #79596 introduced setting `stable` dist tag for backports, but due to #79538 (review), > This sounds like something users would use. However, it might point to 14.x when 15.x exists and we would certainly want people to use 14.x. It's also weird that 14.x would have a `stable` tag but not 15.x. > > That's why `backport` makes more sense since it's less specific and not common vernacular when talking about release characteristics (e.g. "experimental", "nightly", "stable", "unstable" etc.). > > The tag could actually just be "internal-backport" and ideally we'd remove it right after publish. The goal here isn't to come up with a useful tag for backports. We just want to work around NPM requiring a tag during publish. better to use another dist tag: i.e.,`backport`.
1 parent 7347547 commit 3ddc20a

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

scripts/publish-release.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,18 +52,16 @@ const cwd = process.cwd()
5252
// is a backport release. Since NPM sets the 'latest' tag by default
5353
// during publishing, when users install `next@latest`, they might
5454
// get the backported version instead of the actual "latest" version.
55-
// Therefore, we explicitly set the tag as 'stable' for backports.
56-
tag = 'stable'
55+
// Therefore, we explicitly set the tag as 'backport' for backports.
56+
tag = 'backport'
5757
}
5858
}
5959
} catch (error) {
6060
console.log('Failed to fetch Next.js dist tags from the NPM registry.')
6161
throw error
6262
}
6363

64-
console.log(
65-
`Publishing ${isCanary ? 'canary' : isReleaseCandidate ? 'rc' : 'stable'}`
66-
)
64+
console.log(`Publishing as "${tag}" dist tag...`)
6765

6866
if (!process.env.NPM_TOKEN) {
6967
console.log('No NPM_TOKEN, exiting...')

0 commit comments

Comments
 (0)