Skip to content
New issue

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

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

Already on GitHub? # to your account

[#61] Removed --push in favour of --dry-run. #83

Merged
merged 1 commit into from
Mar 30, 2024

Conversation

AlexSkrypnyk
Copy link
Member

@AlexSkrypnyk AlexSkrypnyk commented Mar 30, 2024

closes #61

@codecov-commenter
Copy link

codecov-commenter commented Mar 30, 2024

Codecov Report

Attention: Patch coverage is 16.66667% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 36.83%. Comparing base (d5be94c) to head (0d80cab).

Files Patch % Lines
src/Commands/ArtifactCommand.php 16.66% 5 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   36.83%   36.83%           
=======================================
  Files           6        6           
  Lines         467      467           
=======================================
  Hits          172      172           
  Misses        295      295           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AlexSkrypnyk AlexSkrypnyk merged commit c4c5521 into main Mar 30, 2024
10 checks passed
@AlexSkrypnyk AlexSkrypnyk deleted the feature/remove-push-add-dry-run branch March 30, 2024 04:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove --push in favour of pushing by default
2 participants