-
Notifications
You must be signed in to change notification settings - Fork 444
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
build(deps): bump craft-archives #5124
base: hotfix/8.4
Are you sure you want to change the base?
Conversation
56a1ed5
to
c13c393
Compare
afebe09
to
c136094
Compare
This update fixes LP#2083013, where some package-repositories declarations could conflict with default sources present in Noble and later.
c136094
to
1c11c58
Compare
@@ -25,7 +25,7 @@ codespell==2.3.0 | |||
colorama==0.4.6 | |||
coverage==7.6.1 | |||
craft-application==4.2.7 | |||
craft-archives==2.0.0 | |||
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict |
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.
Just putting this here to make it easy if you take my suggestion about releasing today.
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict | |
craft-archives==2.0.1 |
@@ -20,7 +20,7 @@ charset-normalizer==3.3.2 | |||
click==8.1.7 | |||
colorama==0.4.6 | |||
craft-application==4.2.7 | |||
craft-archives==2.0.0 | |||
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict |
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.
Just putting this here to make it easy if you take my suggestion about releasing today.
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict | |
craft-archives==2.0.1 |
@@ -8,7 +8,7 @@ chardet==5.2.0 | |||
charset-normalizer==3.3.2 | |||
click==8.1.7 | |||
craft-application==4.2.7 | |||
craft-archives==2.0.0 | |||
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict |
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.
Just putting this here to make it easy if you take my suggestion about releasing today.
craft-archives @ git+https://github.com/canonical/craft-archives@work/CRAFT-3538-noble-key-conflict | |
craft-archives==2.0.1 |
@@ -9,6 +9,7 @@ environment: | |||
SNAP/test_multi_keys: test-multi-keys | |||
SNAP/test_foreign_armhf: test-foreign-armhf | |||
SNAP/test_foreign_i386: test-foreign-i386 | |||
SNAP/test_key_conflict: test-key-conflict |
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.
This is an excellent example of well-designed tests sight here.
@@ -0,0 +1,31 @@ | |||
name: test-key-conflict | |||
version: '1.0' | |||
summary: test package repos with keys already present on the system |
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.
Like the well-designed tests above, this is a well-written example that makes it really clear. Thanks!
tests/spread/core24/package-repositories/test-key-conflict/snapcraft.yaml
Show resolved
Hide resolved
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.
Approving with the assumption that @tigarmo will update the requirement files after releasing craft-archives.
This update fixes LP#2083013, where some package-repositories declarations could conflict with default sources present in Noble.
tox run -m lint
?tox run -e test-py310
? (supported versions:py39
,py310
,py311
,py312
)