search

acquia / cli

Acquia CLI
GNU General Public License v2.0
42 stars 47 forks source link

CLI-979: Consolidate update-check logic #1774

Closed danepowell closed 2 weeks ago

danepowell commented 1 month ago

Fix #1403

Waiting on https://github.com/consolidation/self-update/pull/34

danepowell commented 1 month ago

I've added caching to the self-update library and created a new ticket to poll less often in the first place.

Just need to get tests passing now.

github-actions[bot] commented 1 month ago

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1774/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1774/acli.phar
chmod +x acli.phar
codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.21%. Comparing base (402fbef) to head (62b8e53). Report is 3 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1774 +/- ## ============================================ + Coverage 92.02% 92.21% +0.19% + Complexity 1826 1817 -9 ============================================ Files 121 121 Lines 6844 6811 -33 ============================================ - Hits 6298 6281 -17 + Misses 546 530 -16 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.