-
Notifications
You must be signed in to change notification settings - Fork 448
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: use craft-application project variables #4601
feat: use craft-application project variables #4601
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feature/craft-application #4601 +/- ##
=============================================================
+ Coverage 88.83% 89.09% +0.26%
=============================================================
Files 327 332 +5
Lines 22033 22081 +48
=============================================================
+ Hits 19573 19674 +101
+ Misses 2460 2407 -53 ☔ View full report in Codecov by Sentry. |
c1f0300
to
a454ec2
Compare
As far as I can tell, the following tests are now OK regarding the adopt-info part (some are still failing for other reasons):
|
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.
LGTM, please re-request a review from me when the craft-app PR lands
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
a454ec2
to
da4d1bf
Compare
Rebased on |
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
not sure how up-to-date/ready for review this is but the latest spread tests are all broken
oh is this now depending on the craft-app merge? |
The craft-app PR providing this functionality has already landed so I'm not sure what's happening here. |
probably just have to update and re-rerun the tests |
…-adopt-info-and-dynamic-versions
It seems to be good now. |
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.
Spread test failures look as-expected. Thanks!
Define
version
andgrade
as craft-application project variables that canbe updated using craftctl.
tox run -m lint
?tox run -e test-py310
? (supported versions:py39
,py310
,py311
,py312
)