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

ref(commands): Improving commands to use starting services and started #212

Merged
merged 3 commits into from
Jan 16, 2025

Conversation

IanWoodard
Copy link
Member

@IanWoodard IanWoodard commented Jan 15, 2025

https://getsentry.atlassian.net/browse/DEVINFRA-596 Now that we track both starting and started services, commands such as logs and list-services should be able to work with starting services. This change updates those commands to do just that by treating both starting and started services as not-stopped (aka running).

Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.83%. Comparing base (fc673cd) to head (7483aa9).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #212      +/-   ##
==========================================
+ Coverage   92.78%   92.83%   +0.04%     
==========================================
  Files          22       22              
  Lines        1567     1577      +10     
==========================================
+ Hits         1454     1464      +10     
  Misses        113      113              

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

@IanWoodard IanWoodard merged commit bc15b25 into main Jan 16, 2025
14 checks passed
@IanWoodard IanWoodard deleted the iw/improved-commands branch January 16, 2025 01:53
# 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.

2 participants