Remove deprecated flags/options

Author: gustavohenke

bin/index.spec.ts

@@ -288,15 +288,6 @@ describe('--names', () => {
             expect(lines).toContainEqual(expect.stringContaining('[bar] bar'));
         });
     });
-
-    it('is split using --name-separator arg', async () => {
-        const lines = await run(
-            '--names "foo|bar" --name-separator "|" "echo foo" "echo bar"',
-        ).getLogLines();
-
-        expect(lines).toContainEqual(expect.stringContaining('[foo] foo'));
-        expect(lines).toContainEqual(expect.stringContaining('[bar] bar'));
-    });
 });
 
 describe('specifies custom prefix', () => {

bin/index.ts

@@ -4,7 +4,6 @@ import process from 'node:process';
 import yargs from 'yargs';
 import { hideBin } from 'yargs/helpers';
 
-import { assertDeprecated } from '../lib/assert.js';
 import * as defaults from '../lib/defaults.js';
 import { concurrently } from '../lib/index.js';
 import { castArray, splitOutsideParens } from '../lib/utils.js';
@@ -46,12 +45,6 @@ const program = yargs(hideBin(process.argv))
                 'Example names: "main,browser,server"',
             type: 'string',
         },
-        'name-separator': {
-            describe:
-                'The character to split <names> on. Example usage:\n' +
-                '-n "styles|scripts|server" --name-separator "|"',
-            default: defaults.nameSeparator,
-        },
         success: {
             alias: 's',
             describe:
@@ -214,20 +207,7 @@ const program = yargs(hideBin(process.argv))
         },
     })
     .group(
-        [
-            'm',
-            'n',
-            'name-separator',
-            's',
-            'r',
-            'no-color',
-            'hide',
-            'g',
-            'timings',
-            'shell',
-            'P',
-            'teardown',
-        ],
+        ['m', 'n', 's', 'r', 'no-color', 'hide', 'g', 'timings', 'shell', 'P', 'teardown'],
         'General',
     )
     .group(['p', 'c', 'l', 't', 'pad-prefix'], 'Prefix styling')
@@ -237,14 +217,8 @@ const program = yargs(hideBin(process.argv))
     .epilogue(epilogue);
 
 const args = program.parseSync();
-assertDeprecated(
-    args.nameSeparator === defaults.nameSeparator,
-    'name-separator',
-    'Use commas as name separators instead.',
-);
 
-// Get names of commands by the specified separator
-const names = (args.names || '').split(args.nameSeparator);
+const names = (args.names || '').split(',');
 
 const additionalArguments = castArray(args['--'] ?? []).map(String);
 const commands = args.passthroughArguments ? args._ : args._.concat(additionalArguments);

lib/defaults.ts

@@ -21,11 +21,6 @@ export const maxProcesses = 0;
  */
 export const hide = '';
 
-/**
- * The character to split <names> on.
- */
-export const nameSeparator = ',';
-
 /**
  * Which prefix style to use when logging processes output.
  */

lib/index.ts

@@ -1,7 +1,7 @@
 import process from 'node:process';
 import { Readable } from 'node:stream';
 
-import { assertDeprecated, assertNotRuntime } from './assert.js';
+import { assertNotRuntime } from './assert.js';
 import { CloseEvent, Command, CommandIdentifier, TimerEvent } from './command.js';
 import {
     concurrently as createConcurrently,
@@ -84,11 +84,6 @@ export type ConcurrentlyOptions = Omit<
     restartTries?: number;
 
     // Process killing options
-    /**
-     * @deprecated Use `killOthersOn` instead.
-     * @see KillOthers
-     */
-    killOthers?: ProcessCloseCondition | ProcessCloseCondition[];
     /**
      * Once the first command exits with one of these statuses, kill other commands.
      * @see KillOthers
@@ -137,7 +132,6 @@ export function concurrently(
     commands: ConcurrentlyCommandInput[],
     options: Partial<ConcurrentlyOptions> = {},
 ) {
-    assertDeprecated(options.killOthers === undefined, 'killOthers', 'Use killOthersOn instead.');
     assertNotRuntime(
         // When run via /snap/bin/node, process.execPath maps to the actual snap path, but it also sets
         // several SNAP_* env variables. If the snap is run directly via e.g. /snap/node/current/bin/node,
@@ -201,7 +195,7 @@ export function concurrently(
             }),
             new KillOthers({
                 logger,
-                conditions: options.killOthersOn || options.killOthers || [],
+                conditions: options.killOthersOn || [],
                 timeoutMs: options.killTimeout,
                 killSignal: options.killSignal,
                 abortController,