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

Feature: Synchronise all packages command #536

Merged
merged 3 commits into from
Jan 6, 2022

Conversation

xvilo
Copy link
Contributor

@xvilo xvilo commented Jan 5, 2022

This PR adds the ability to sync all packages at once. We currently have the need for this sometimes and are already able to do this manually through the API. However, it's much easier if this is an out-of-the-box CLI feature.

You can specify to sync all packages from an Organization, or just go through all Organizations and sync their packages.

Can be called like:

# Synchronise all packages from the "buddy" Organization
$ ./bin/console repman:package:synchronize-all buddy

# Synchronise all packages from all available Organizations
$ ./bin/console repman:package:synchronize-all
Synchronizing packages of 2 organizations.
Synchronizing packages for test.
- xvilo/harvest-forceast
Synchronizing packages for buddy.
- cache/filesystem-adapter
- symfony/var-dumper
- symfony/translation
- symfony/routing

Will throw an InvalidArgumentException if the supplied Organization Alias can not be found.

@xvilo xvilo force-pushed the feature/syncronize-all-packages branch from e8e2a90 to 41f6624 Compare January 5, 2022 10:13
@xvilo xvilo force-pushed the feature/syncronize-all-packages branch from 41f6624 to 133e3d3 Compare January 5, 2022 10:15
@codecov
Copy link

codecov bot commented Jan 5, 2022

Codecov Report

Merging #536 (8307899) into master (0de28d9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #536   +/-   ##
=========================================
  Coverage     99.33%   99.33%           
- Complexity     1885     1893    +8     
=========================================
  Files           300      301    +1     
  Lines          5973     5996   +23     
=========================================
+ Hits           5933     5956   +23     
  Misses           40       40           
Impacted Files Coverage Δ
src/Command/SynchronizeAllPackagesCommand.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0de28d9...8307899. Read the comment docs.

Copy link
Member

@akondas akondas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thanks 🍻

@akondas akondas merged commit 322570d into repman-io:master Jan 6, 2022
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

Successfully merging this pull request may close these issues.

2 participants