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

fix: update to craft-application latest #4659

Merged
merged 4 commits into from
Mar 13, 2024

Conversation

mr-cal
Copy link
Collaborator

@mr-cal mr-cal commented Mar 13, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

#4616, #4643, and #4651 all accommodate for different breaking changes in craft-application's main branch.

This PR has a commit for each of those PRs.

mr-cal and others added 3 commits March 13, 2024 08:12
Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Create a default build plan with a build info matching the host's architecture
and base, so that individual plan-dependent tests don't have to worry about
tweaking the plan.
@mr-cal mr-cal added the rebase label Mar 13, 2024
Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Copy link

codecov bot commented Mar 13, 2024

Codecov Report

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

Project coverage is 89.22%. Comparing base (a019fe7) to head (fda349b).
Report is 25 commits behind head on feature/craft-application.

Files Patch % Lines
snapcraft/commands/unimplemented.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/craft-application    #4659      +/-   ##
=============================================================
+ Coverage                      89.14%   89.22%   +0.07%     
=============================================================
  Files                            331      334       +3     
  Lines                          22078    22277     +199     
=============================================================
+ Hits                           19682    19876     +194     
- Misses                          2396     2401       +5     

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

@mr-cal mr-cal requested a review from tigarmo March 13, 2024 13:39
@mr-cal
Copy link
Collaborator Author

mr-cal commented Mar 13, 2024

Spread test failures look as expected to me.

The only "new" failure I noticed is that tests/spread/core24/remove-hook no longer fails on --use-lxd in snapcraft --use-lxd but fails later on in the test. I haven't looked into it.

@mr-cal
Copy link
Collaborator Author

mr-cal commented Mar 13, 2024

@sergiusens - can you land this?

@sergiusens sergiusens merged commit b601c77 into feature/craft-application Mar 13, 2024
7 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants