chore: don't deprecate lax on the config commands, and ammend the message that is logged in the console for the deprecations

Signed-off-by: blam <ben@blam.sh>

packages/cli/src/commands/app/build.ts

@@ -26,7 +26,9 @@ export default async (cmd: Command) => {
   if (cmd.lax) {
     console.warn(
       chalk.yellow(
-        '[DEPRECATED] - The lax argument is deprecated and will be removed in the future.',
+        `[DEPRECATED] - The --lax option is deprecated and will be removed in the future.
+        Please open an issue towards https://github.com/backstage/backstage that describes your use-case if you need the flag to stay around
+    `,
       ),
     );
   }

packages/cli/src/commands/config/print.ts

@@ -19,17 +19,8 @@ import { stringify as stringifyYaml } from 'yaml';
 import { AppConfig, ConfigReader } from '@backstage/config';
 import { loadCliConfig } from '../../lib/config';
 import { ConfigSchema, ConfigVisibility } from '@backstage/config-loader';
-import chalk from 'chalk';
 
 export default async (cmd: Command) => {
-  if (cmd.lax) {
-    console.warn(
-      chalk.yellow(
-        '[DEPRECATED] - The lax argument is deprecated and will be removed in the future.',
-      ),
-    );
-  }
-
   const { schema, appConfigs } = await loadCliConfig({
     args: cmd.config,
     fromPackage: cmd.package,

packages/cli/src/commands/config/validate.ts

@@ -14,18 +14,10 @@
  * limitations under the License.
  */
 
-import chalk from 'chalk';
 import { Command } from 'commander';
 import { loadCliConfig } from '../../lib/config';
 
 export default async (cmd: Command) => {
-  if (cmd.lax) {
-    console.warn(
-      chalk.yellow(
-        '[DEPRECATED] - The lax argument is deprecated and will be removed in the future.',
-      ),
-    );
-  }
   await loadCliConfig({
     args: cmd.config,
     fromPackage: cmd.package,

packages/cli/src/commands/index.ts

@@ -30,7 +30,6 @@ export function registerCommands(program: CommanderStatic) {
     .command('app:build')
     .description('Build an app for a production release')
     .option('--stats', 'Write bundle stats to output directory')
-    // Deprecated
     .option(
       '--lax',
       '[DEPRECATED] - Do not require environment variables to be set',
@@ -172,10 +171,7 @@ export function registerCommands(program: CommanderStatic) {
       '--package <name>',
       'Only load config schema that applies to the given package',
     )
-    .option(
-      '--lax',
-      '[DEPRECATED] - Do not require environment variables to be set',
-    )
+    .option('--lax', 'Do not require environment variables to be set')
     .option('--frontend', 'Print only the frontend configuration')
     .option('--with-secrets', 'Include secrets in the printed configuration')
     .option(
@@ -192,10 +188,7 @@ export function registerCommands(program: CommanderStatic) {
       '--package <name>',
       'Only load config schema that applies to the given package',
     )
-    .option(
-      '--lax',
-      '[DEPRECATED] - Do not require environment variables to be set',
-    )
+    .option('--lax', 'Do not require environment variables to be set')
     .option('--frontend', 'Only validate the frontend configuration')
     .option(...configOption)
     .description(