-
-
Notifications
You must be signed in to change notification settings - Fork 9
Release 6.0.0 #47
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
Release 6.0.0 #47
Conversation
btkostner
left a comment
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.
Changing this branch to 6.0.0 means every day we would be publishing a new 6 version. We should probably keep this as daily so our cron just builds the daily branch, and we only release a new 6 version when we create a release
|
I think that still makes sense to keep the version number up to date. So if we make a change we'd bump the version number in daily Otherwise we would publish a runtime to the |
|
Daily GitHub action doesn't have a hard set branch, so it will use the version in the json file. So by changing this, you're making a new branch of If you want, we can change the Daily GitHub action to set the branch to As far as I know, there is no difference between |
|
Ah yeah okay that's kind of confusing but yeah we should make sure the daily publish action is pushing to When you run For example the version of FreeDesktop platform I have is "20.08.14" and the branch is "20.08" |
btkostner
left a comment
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 hard sets daily to always be the daily branch ✔️
Will publish releases to the 6 branch ✔️
Not 100% sure that the branch: 6.0.0 will have any effect
|
Prior to merging, let's poke @Marukesu to see if they have clarifying thoughts on the version stuff :) |
About the version column in |
|
I updated the manifest to |
|
Okay so it sounds like we need to add AppData here as well. I can take care of that today |
btkostner
left a comment
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 think this looks good to me
Uh oh!
There was an error while loading. Please reload this page.