Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the ability to sync the recipe of a single package #501

Closed
tsantos84 opened this issue May 15, 2019 · 9 comments
Closed

Add the ability to sync the recipe of a single package #501

tsantos84 opened this issue May 15, 2019 · 9 comments

Comments

@tsantos84
Copy link

tsantos84 commented May 15, 2019

Currently sync-recipes syncs all packages that have recipes and can be dangerous if I need to sync only one recipe. A useful improvement would be sync a single recipe so we can have more control in what are being updated. To support that, a separated command to list all packages configured by recipes could be added in this feature, like this:

composer recipe:list which would print:

  • symfony/console [oficial]
  • symfony/validator [oficial]
  • tsantos/serializer-bundle [contrib]

and now is possible to sync one or more package:

composer sync-recipes symfony/console tsantos/serializer-bundle

The same above command could be renamed to:

composer recipe:sync symfony/console tsantos/serializer-bundle

I'm not sure if this command should list packages names or recipes names.

Following this way, we could have a namespace called recipe to hold all commands related to recipes.

  • recipe:list
  • recipe:sync
  • recipe:fix
@tsantos84 tsantos84 changed the title Add the ability to sync the recipe of a single recipe Add the ability to sync the recipe of a single package May 15, 2019
@Pierstoval
Copy link
Contributor

Why would one need composer recipe:list when you can just see this in symfony.lock quite clearly?

@keichinger
Copy link

@Pierstoval it's way more convenient and could also be used for some statistics/analytics that are gathered by your CI. You could make the same argument about almost any other console command.

@tsantos84
Copy link
Author

As @Pierstoval said, it's better to run a command and see the list of recipes rather than open the symfomy.lock file.

@nicolas-grekas
Copy link
Member

nicolas-grekas commented Jun 5, 2019

I wouldn't change the existing names as that's a too hard road for existing teaching materials, e.g. documentation, slides, tutorials.
But having a --dry-run option would be great, and being able to specify a list of packages on the existing command too.

@tsantos84
Copy link
Author

What about to keep the original name as alias to the new ones?

@nicolas-grekas
Copy link
Member

The new names would need to be significantly better, but I'm not sure they are personally :)

@maxhelias
Copy link
Contributor

Look at the RFC #518 your need is listed

tgalopin pushed a commit to tgalopin/flex that referenced this issue Dec 3, 2020
@nicolas-grekas
Copy link
Member

Closing in favor of #501

@maxhelias
Copy link
Contributor

Rather : #562

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

No branches or pull requests

5 participants