paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 46 Packages Projects Releases Wiki Activity

Merge pull request #33181 from backstage/rugvip/cli-boolean-flag-negation

Browse Source 
cli: enable --no- prefix negation for boolean flags
This commit is contained in:
Patrik Oldsberg
2026-03-10 18:27:53 +01:00
committed by GitHub
parent 106d1b2a0e 74aa55f0e9
commit 1097f68579
32 changed files with 33 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/package.json
+1 -1
View File
@@ -81,7 +81,7 @@
"buffer": "^6.0.3",
"chalk": "^4.0.0",
"chokidar": "^3.3.1",
"cleye": "^2.2.1",
"cleye": "^2.3.0",
"commander": "^14.0.3",
"cross-fetch": "^4.0.0",
"cross-spawn": "^7.0.3",
packages/cli/src/modules/build/commands/buildWorkspace.ts
+1
View File
@@ -38,6 +38,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: { ...info, usage: `${info.usage} <workspace-dir> [packages...]` },
booleanFlagNegation: true,
parameters: ['<workspace-dir>', '[packages...]'],
flags: {
alwaysPack: {
packages/cli/src/modules/build/commands/package/build/command.ts
+1
View File
@@ -44,6 +44,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
role: {
type: String,
packages/cli/src/modules/build/commands/package/clean.ts
+1 -1
View File
@@ -20,7 +20,7 @@ import { targetPaths } from '@backstage/cli-common';
import type { CommandContext } from '../../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
await fs.remove(targetPaths.resolve('dist'));
await fs.remove(targetPaths.resolve('dist-types'));
await fs.remove(targetPaths.resolve('coverage'));
packages/cli/src/modules/build/commands/package/postpack.ts
+1 -1
View File
@@ -20,6 +20,6 @@ import { revertProductionPack } from '../../lib/packager/productionPack';
import type { CommandContext } from '../../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
await revertProductionPack(targetPaths.dir);
};
packages/cli/src/modules/build/commands/package/prepack.ts
+1 -1
View File
@@ -23,7 +23,7 @@ import { createTypeDistProject } from '../../lib/typeDistProject';
import type { CommandContext } from '../../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
publishPreflightCheck({
dir: targetPaths.dir,
packages/cli/src/modules/build/commands/package/start/command.ts
+1
View File
@@ -36,6 +36,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
config: {
type: [String],
packages/cli/src/modules/build/commands/repo/build.ts
+1
View File
@@ -37,6 +37,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
all: {
type: Boolean,
packages/cli/src/modules/build/commands/repo/clean.ts
+1 -1
View File
@@ -22,7 +22,7 @@ import { run, targetPaths } from '@backstage/cli-common';
import type { CommandContext } from '../../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
const packages = await PackageGraph.listTargetPackages();
await fs.remove(targetPaths.resolveRoot('dist'));
packages/cli/src/modules/build/commands/repo/start.ts
+1
View File
@@ -42,6 +42,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: { ...info, usage: `${info.usage} [packages...]` },
booleanFlagNegation: true,
parameters: ['[packages...]'],
flags: {
plugin: {
packages/cli/src/modules/config/commands/docs.ts
+1
View File
@@ -31,6 +31,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
package: {
type: String,
packages/cli/src/modules/config/commands/print.ts
+1
View File
@@ -27,6 +27,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
package: { type: String, description: 'Package to print config for' },
lax: {
packages/cli/src/modules/config/commands/schema.ts
+1
View File
@@ -28,6 +28,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
package: { type: String, description: 'Package to print schema for' },
format: { type: String, description: 'Output format (yaml or json)' },
packages/cli/src/modules/config/commands/validate.ts
+1
View File
@@ -24,6 +24,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
package: {
type: String,
packages/cli/src/modules/create-github-app/commands/create-github-app/index.ts
+1
View File
@@ -32,6 +32,7 @@ export default async ({ args, info }: CommandContext) => {
const { _: positionals } = cli(
{
help: { ...info, usage: `${info.usage} <github-org>` },
booleanFlagNegation: true,
parameters: ['<github-org>'],
},
undefined,
packages/cli/src/modules/info/commands/info.ts
+1
View File
@@ -58,6 +58,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
include: {
type: [String],
packages/cli/src/modules/lint/commands/package/lint.ts
+1
View File
@@ -27,6 +27,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: { ...info, usage: `${info.usage} [directories...]` },
booleanFlagNegation: true,
parameters: ['[directories...]'],
flags: {
fix: {
packages/cli/src/modules/lint/commands/repo/lint.ts
+1
View File
@@ -66,6 +66,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
fix: {
type: Boolean,
packages/cli/src/modules/maintenance/commands/repo/fix.ts
+1
View File
@@ -501,6 +501,7 @@ export default async ({
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
publish: {
type: Boolean,
packages/cli/src/modules/maintenance/commands/repo/list-deprecations.ts
+1
View File
@@ -28,6 +28,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
json: { type: Boolean, description: 'Output as JSON' },
},
packages/cli/src/modules/migrate/commands/packageExports.ts
+1 -1
View File
@@ -18,7 +18,7 @@ import { cli } from 'cleye';
import type { CommandContext } from '../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
throw new Error(
'The `migrate package-exports` command has been removed, use `repo fix` instead.',
);
packages/cli/src/modules/migrate/commands/packageLintConfigs.ts
+1 -1
View File
@@ -24,7 +24,7 @@ import type { CommandContext } from '../../../wiring/types';
const PREFIX = `module.exports = require('@backstage/cli/config/eslint-factory')`;
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
const packages = await PackageGraph.listTargetPackages();
const oldConfigs = [
packages/cli/src/modules/migrate/commands/packageRole.ts
+1 -1
View File
@@ -23,7 +23,7 @@ import { targetPaths } from '@backstage/cli-common';
import type { CommandContext } from '../../../wiring/types';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
const { packages } = await getPackages(targetPaths.dir);
await Promise.all(
packages/cli/src/modules/migrate/commands/packageScripts.ts
+1 -1
View File
@@ -25,7 +25,7 @@ const configArgPattern = /--config[=\s][^\s$]+/;
const noStartRoles: PackageRole[] = ['cli', 'common-library'];
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
const packages = await PackageGraph.listTargetPackages();
await Promise.all(
packages/cli/src/modules/migrate/commands/reactRouterDeps.ts
+1 -1
View File
@@ -24,7 +24,7 @@ const REACT_ROUTER_DEPS = ['react-router', 'react-router-dom'];
const REACT_ROUTER_RANGE = '6.0.0-beta.0 || ^6.3.0';
export default async ({ args, info }: CommandContext) => {
cli({ help: info }, undefined, args);
cli({ help: info, booleanFlagNegation: true }, undefined, args);
const packages = await PackageGraph.listTargetPackages();
await Promise.all(
packages/cli/src/modules/migrate/commands/versions/bump.ts
+1
View File
@@ -80,6 +80,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
pattern: {
type: String,
packages/cli/src/modules/migrate/commands/versions/migrate.ts
+1
View File
@@ -45,6 +45,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
pattern: {
type: String,
packages/cli/src/modules/new/commands/new.ts
+1
View File
@@ -41,6 +41,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
select: {
type: String,
packages/cli/src/modules/test/commands/repo/test.ts
+1
View File
@@ -147,6 +147,7 @@ export default async ({ args, info }: CommandContext) => {
const { flags: opts } = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
since: {
type: String,
packages/cli/src/modules/translations/commands/export.ts
+1
View File
@@ -41,6 +41,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
output: {
type: String,
packages/cli/src/modules/translations/commands/import.ts
+1
View File
@@ -47,6 +47,7 @@ export default async ({ args, info }: CommandContext) => {
} = cli(
{
help: info,
booleanFlagNegation: true,
flags: {
input: {
type: String,