ci(release): handling for create-discord-app (#11143)

* ci(release): handling for create-discord-app

* ci(deprecate): cda support

* ci: update our custom action to handle the renaming when invoked

* fix: don't double release on github

* chore: just in case

---------

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
This commit is contained in:
Denis-Adrian Cristea
2025-10-08 13:33:51 +03:00
committed by GitHub
parent aac247cc18
commit fcf7f27fd7
2 changed files with 12 additions and 2 deletions

View File

@@ -11,6 +11,7 @@ on:
- '@discordjs/builders' - '@discordjs/builders'
- '@discordjs/collection' - '@discordjs/collection'
- '@discordjs/core' - '@discordjs/core'
- 'create-discord-app'
- 'create-discord-bot' - 'create-discord-bot'
- '@discordjs/formatters' - '@discordjs/formatters'
- 'discord.js' - 'discord.js'

View File

@@ -41,7 +41,7 @@ async function gitTagAndRelease(release: ReleaseEntry, dry: boolean) {
} }
} }
export async function releasePackage(release: ReleaseEntry, dev: boolean, dry: boolean) { export async function releasePackage(release: ReleaseEntry, dev: boolean, dry: boolean, doGitRelease = !dev) {
// Sanity check against the registry first // Sanity check against the registry first
if (await checkRegistry(release)) { if (await checkRegistry(release)) {
info(`${release.name}@${release.version} already published, skipping.`); info(`${release.name}@${release.version} already published, skipping.`);
@@ -54,7 +54,8 @@ export async function releasePackage(release: ReleaseEntry, dev: boolean, dry: b
await $`pnpm --filter=${release.name} publish --provenance --no-git-checks ${dev ? '--tag=dev' : ''}`; await $`pnpm --filter=${release.name} publish --provenance --no-git-checks ${dev ? '--tag=dev' : ''}`;
} }
if (!dev) await gitTagAndRelease(release, dry); // && !dev just to be sure
if (doGitRelease && !dev) await gitTagAndRelease(release, dry);
if (dry) return; if (dry) return;
@@ -83,4 +84,12 @@ export async function releasePackage(release: ReleaseEntry, dev: boolean, dry: b
// eslint-disable-next-line promise/prefer-await-to-then // eslint-disable-next-line promise/prefer-await-to-then
.then(() => {}); .then(() => {});
} }
// Evil, but I can't think of a cleaner mechanism
if (release.name === 'create-discord-bot') {
await $`pnpm --filter=create-discord-bot run rename-to-app`;
// eslint-disable-next-line require-atomic-updates
release.name = 'create-discord-app';
await releasePackage(release, dev, dry, false);
}
} }