Skip to content

ci: Ensure all apt install have -y#16655

Open
sergio-costas wants to merge 1 commit intocanonical:masterfrom
sergio-costas:ensure-all-apt-install-have-yes-option
Open

ci: Ensure all apt install have -y#16655
sergio-costas wants to merge 1 commit intocanonical:masterfrom
sergio-costas:ensure-all-apt-install-have-yes-option

Conversation

@sergio-costas
Copy link
Contributor

During tests, a lot of .deb packages are installed. In some cases, the -y parameter is missing, which may cause problems in the future if the package to install have new dependencies.

This patch ensures that all apt install calls do have the -y parameter.

Thanks for helping us make a better snapd!
Have you signed the license agreement and read the contribution guide?

During tests, a lot of .deb packages are installed. In some
cases, the -y parameter is missing, which may cause problems in
the future if the package to install have new dependencies.

This patch ensures that all apt install calls do have the -y
parameter.
@sergio-costas sergio-costas force-pushed the ensure-all-apt-install-have-yes-option branch from 959fcb0 to 6ddc1a1 Compare February 24, 2026 11:51
@github-actions
Copy link

github-actions bot commented Feb 24, 2026

Tue Feb 24 13:12:10 UTC 2026
The following results are from: https://github.com/canonical/snapd/actions/runs/22349596011

No spread failures reported

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.50%. Comparing base (fd20faf) to head (6ddc1a1).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #16655   +/-   ##
=======================================
  Coverage   77.50%   77.50%           
=======================================
  Files        1354     1353    -1     
  Lines      186441   186437    -4     
  Branches     2449     2449           
=======================================
- Hits       144510   144507    -3     
+ Misses      33195    33192    -3     
- Partials     8736     8738    +2     
Flag Coverage Δ
unittests 77.50% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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