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
Lock snapcraft to v5 since v7 breaks the build #259
Conversation
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, should fix. Though, we should try to fix it for v7.x when we have the time
@@ -261,7 +261,7 @@ jobs: | |||
CSC_IDENTITY_AUTO_DISCOVERY: false | |||
SNAPCRAFT_LOGIN: ${{ secrets.SNAPCRAFT_LOGIN }} | |||
run: | | |||
sudo snap install snapcraft --classic | |||
sudo snap install snapcraft --channel=5.x/stable --classic |
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.
it should be v6.1 as per this successful build https://github.com/ferdium/ferdium-app/runs/6872922102?check_suite_focus=true#step:16:20
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.
V6 isnt listed at https://snapcraft.io/snapcraft
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.
V6 isnt listed at https://snapcraft.io/snapcraft
Weird :o
@NathanaelGandhi , by any chance do you have time to adapt our build to 7.x today?
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.
Yes - this page doesn't list any 6.x
- thus, I left it at 5.x
@@ -273,7 +273,7 @@ jobs: | |||
CSC_IDENTITY_AUTO_DISCOVERY: false | |||
SNAPCRAFT_LOGIN: ${{ secrets.SNAPCRAFT_LOGIN }} | |||
run: | | |||
sudo snap install snapcraft --classic | |||
sudo snap install snapcraft --channel=5.x/stable --classic |
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.
it should be v6.1 as per this successful build https://github.com/ferdium/ferdium-app/runs/6872922102?check_suite_focus=true#step:16:20
@@ -285,7 +285,7 @@ jobs: | |||
CSC_IDENTITY_AUTO_DISCOVERY: false | |||
SNAPCRAFT_LOGIN: ${{ secrets.SNAPCRAFT_LOGIN }} | |||
run: | | |||
sudo snap install snapcraft --classic | |||
sudo snap install snapcraft --channel=5.x/stable --classic |
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.
it should be v6.1 as per this successful build https://github.com/ferdium/ferdium-app/runs/6872922102?check_suite_focus=true#step:16:20
versions available at: https://snapcraft.io/snapcraft
versions available at: https://snapcraft.io/snapcraft
Pre-flight Checklist
Description of Change
Trying to fix the build by locking on v5.x (prior to v7, the latest version available is v5: https://snapcraft.io/snapcraft)
Motivation and Context
Get successful nightly builds.
Screenshots
Checklist
npm run prepare-code
)npm test
passesRelease Notes