-
Notifications
You must be signed in to change notification settings - Fork 11
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
ci: use common snapcrafters ci; update README #62
Conversation
@ThyMYthOS can you take a look at this? I can't seem to tag you as a reviewer 🤷♂️ |
ae05926
to
826cda2
Compare
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.
Will the new candidate
branch work together with what is proposed here #10 ?
@@ -0,0 +1,25 @@ | |||
name: Promote |
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.
Not sure, if we should promote to stable via GitHub, since we don't have proper regression tests to do this automatically.
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.
The promotion would only happen by one of the maintainers issuing a slash command - and after the app is launched and automatically screenshotted to ensure it launches
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.
See here for an example: snapcrafters/signal-desktop#269
Yes! This "candidate" branch will map to the "latest" track in the store, but we can create additional branches/tracks |
Adds remaining snapcrafters workflows and updates the README to the latest template.