Skip to content
This repository has been archived by the owner on Mar 6, 2021. It is now read-only.

Command "normalize" is not defined #80

Closed
spawnia opened this issue Apr 18, 2020 · 3 comments · Fixed by #81
Closed

Command "normalize" is not defined #80

spawnia opened this issue Apr 18, 2020 · 3 comments · Fixed by #81

Comments

@spawnia
Copy link

spawnia commented Apr 18, 2020

Steps required to reproduce the problem

  1. Use the first example provided in the config

Expected Result

  • The action should run successfully

Actual Result

  • Command "normalize" is not defined

See https://github.com/nuwave/lighthouse/pull/1299/checks?check_run_id=597704867

@spawnia
Copy link
Author

spawnia commented May 4, 2020

Thanks @localheinz, the current :latest is working for me. Awaiting a release 👍

@localheinz
Copy link
Member

@spawnia

Apologies for the delay!

In the meantime, I archived the repository, then unarchived again and fixed the issue. Note, though, that I made some unexpected changes that might make work necessary on your end:

By default, I'm not running

$ composer normalize

with the --dry-run option anymore. If you need this, you now need to specify arguments and options!

@spawnia
Copy link
Author

spawnia commented May 4, 2020

Do you couple that with an option to automatically commit the changes?

I have had a great time running Prettier that way, it would fit here as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants