composer-normalize icon indicating copy to clipboard operation
composer-normalize copied to clipboard

Pass composer parameters to update subcommand

Open xurizaemon opened this issue 3 years ago • 1 comments

Motivation

ANSI or no-ANSI output is relatively trivial, but I would expect composer --foo normalize to pass --foo on to any Composer subcommands. Currently I believe it does not, and propose that we change this.

There may be Other Considerations, I'm not strongly wedded to this change :)

We might want to include or exclude specific composer switches to retain existing behaviour (eg --no-update-lock behaviour should not change?).

Steps required to reproduce the problem

  1. Run composer --no-ansi normalize, eg in a CI process
  2. See ANSI output
cd "./build/ci-basic"; composer normalize
Running ergebnis/composer-normalize by Andreas Möller and contributors.
Successfully normalized ./composer.json.
Updating lock file.
Loading composer repositories with package information
Updating dependencies
Nothing to modify in lock file
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals
    0 [>---------------------------]
    0 [>---------------------------]
    0 [>---------------------------]
  - Installing drupal/core (9.1.8): Extracting archive
  - Installing drupal/raven (3.2.1): Extracting archive
 0/2 [>---------------------------]   0%
 1/2 [==============>-------------]  50%
 2/2 [============================] 100%Package container-interop/container-interop is abandoned, you should avoid using it. Use psr/container instead.
Package doctrine/reflection is abandoned, you should avoid using it. Use roave/better-reflection instead.
91 packages you are using are looking for funding.
Use the `composer fund` command to find out more!

Expected Result

  • Don't show ANSI output, since the parent composer command did not

Actual Result

    0 [>---------------------------]
    0 [>---------------------------]
    0 [>---------------------------]
  - Installing drupal/core (9.1.8): Extracting archive
  - Installing drupal/raven (3.2.1): Extracting archive
 0/2 [>---------------------------]   0%
 1/2 [==============>-------------]  50%
 2/2 [============================] 100%

xurizaemon avatar May 17 '21 01:05 xurizaemon

Since this issue has not had any activity within the last 180 days, I have marked it as stale.

I will close it if no further activity occurs within the next 14 days.

ergebnis-bot avatar Mar 31 '22 12:03 ergebnis-bot

Since this issue has not had any activity within the last 180 days, I have marked it as stale.

I will close it if no further activity occurs within the next 14 days.

ergebnis-bot avatar Sep 27 '22 12:09 ergebnis-bot

Thanks!

xurizaemon avatar May 03 '23 18:05 xurizaemon

You are welcome, @xurizaemon!

localheinz avatar May 03 '23 20:05 localheinz