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

Update UpdateConfigsCommand to prompt for which config tag to publish #984

Conversation

caendesilva
Copy link
Member

Takes advantage of the tags introduced in #983 to allow user to select to only publish Hyde configs, Laravel configs, or both (default).

@codecov
Copy link

codecov bot commented Feb 12, 2023

Codecov Report

Merging #984 (4c70fff) into separate-internal-application-files-from-project-structure- (00a6fd0) will not change coverage.
The diff coverage is n/a.

@@                                       Coverage Diff                                       @@
##             separate-internal-application-files-from-project-structure-      #984   +/-   ##
===============================================================================================
  Coverage                                                         100.00%   100.00%           
- Complexity                                                          1270      1271    +1     
===============================================================================================
  Files                                                                151       151           
  Lines                                                               3357      3367   +10     
===============================================================================================
+ Hits                                                                3357      3367   +10     
Impacted Files Coverage Δ
...work/src/Console/Commands/UpdateConfigsCommand.php 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@caendesilva caendesilva marked this pull request as ready for review February 12, 2023 14:24
@caendesilva caendesilva merged commit ab1b414 into separate-internal-application-files-from-project-structure- Feb 12, 2023
@caendesilva caendesilva changed the title Update UpdateConfigsCommand to prompt for which tag to publish Update UpdateConfigsCommand to prompt for which config tag to publish Feb 12, 2023
@caendesilva caendesilva deleted the update-publishable-config-choices-to-allow-specification-of-which-group-to-publish branch February 12, 2023 14:25
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.

1 participant