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

feat: improve lifecycle messages #4431

Merged
merged 3 commits into from Nov 13, 2023
Merged

feat: improve lifecycle messages #4431

merged 3 commits into from Nov 13, 2023

Conversation

sergiusens
Copy link
Collaborator

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run make lint?
  • Have you successfully run pytest tests/unit?

Copy link
Contributor

@cmatsuoka cmatsuoka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to have similar messages in craft-application as well to make the transition smoother.

@sergiusens
Copy link
Collaborator Author

@cmatsuoka I copied them from craft application!

@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2023

Codecov Report

Merging #4431 (ba00727) into main (c7ebc34) will increase coverage by 0.00%.
The diff coverage is 87.50%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##             main    #4431   +/-   ##
=======================================
  Coverage   89.17%   89.18%           
=======================================
  Files         321      321           
  Lines       21658    21655    -3     
=======================================
- Hits        19314    19312    -2     
+ Misses       2344     2343    -1     
Files Coverage Δ
snapcraft/parts/parts.py 81.34% <87.50%> (-0.41%) ⬇️

... and 1 file with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
@sergiusens sergiusens merged commit 904d0b2 into main Nov 13, 2023
9 of 10 checks passed
@sergiusens sergiusens deleted the better-messages branch November 13, 2023 12:27
syu-w pushed a commit that referenced this pull request Jan 29, 2024
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
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.

None yet

5 participants