-
Notifications
You must be signed in to change notification settings - Fork 562
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
tests: clean snaps.sh helper #10343
tests: clean snaps.sh helper #10343
Conversation
…ps.sh The change includes: . Start using the tool snaps-state . Remove includes for snaps.sh . Cleanup snaps.sh helper . Update snaps-state too to include repack with target dir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, just one (nitpicky) question inline
@@ -88,7 +85,7 @@ execute: | | |||
if [ "$NESTED_BUILD_SNAPD_FROM_CURRENT" = "false" ]; then | |||
if os.query is-xenial; then | |||
# build the core snap for this run | |||
repack_snapd_deb_into_core_snap "$PWD" | |||
"$TESTSTOOLS"/snaps-state repack_snapd_deb_into_snap core "$PWD" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we could remove "$PWD" here as it's the default?
Codecov Report
@@ Coverage Diff @@
## master #10343 +/- ##
==========================================
- Coverage 80.71% 78.31% -2.40%
==========================================
Files 727 882 +155
Lines 58158 99308 +41150
==========================================
+ Hits 46940 77777 +30837
- Misses 7544 16638 +9094
- Partials 3674 4893 +1219
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
The change includes:
. Start using the tool snaps-state
. Remove includes for snaps.sh
. Cleanup snaps.sh helper
. Update snaps-state too to include repack with target dir