-
Notifications
You must be signed in to change notification settings - Fork 79
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
Wrong version displayed in night builds #12349
Comments
Yeah we stopped incrementing the VERSION number in master; instead the git hash should be shown... I wonder what's different here. |
I think @caybro fixed this one lately? |
Ehm, that was the status-go version; ours is (will be) still wrong probably... :/ |
Either we update the VERSION file manually with each release, to be something "higher", or we need to figure out some generic "git-like version" |
I'll try to think about something |
Yeah, manually tagging master after each release, or manually increasing the VERSION number; I don't see much difference tbh 😄 |
Something like this, yes. When starting a Something like this, but needs to be tested. And also this sounds a bit overcomplicated :D It will be a manual process, but I don't think that's a problem. |
- master should always have an up-to-date version, greater than the current stable version - for now this is 2.30.90; as 2.30 gets released, we bump it again to 2.31.90, and so on Fixes #12349
I find it weird to see How
|
lol, turned out that I already proposed this in the comment above: #12349 (comment) 😄 |
Yeah I like the idea too, and I said it even before 😆. This will just require some manual intervention no matter what; in the end it doesn't matter much if it's editing |
In current master:
|
Oh and turns out if I do this:
it will append the |
Last remark, the only thing I'm unsure (and quite frankly the only downside) is that only @iurimatias and maybe @jrainville are able to push git tags? |
This just shows that you have some local uncommitted changes. |
In status-go anyone can push tags. So I think it's the same in desktop. |
- remove VERSION file - some smaller cleanups and warning fixes Fixes #12349
- remove VERSION file - some smaller cleanups and warning fixes Fixes #12349
- remove VERSION file - some smaller cleanups and warning fixes Fixes #12349
- remove VERSION file - expose the `GIT_COMMIT`so that we can properly construct the web links when the user clicks the version number in Settings/About - some smaller cleanups and warning fixes Fixes #12349
- remove VERSION file - expose the `GIT_COMMIT`so that we can properly construct the web links when the user clicks the version number in Settings/About - some smaller cleanups and warning fixes Fixes #12349
Bug Report
Images downloaded from here: https://ci.infra.status.im/job/status-desktop/job/master/ shows
0.9.90
version instead of the actual one.It seems to be coming from here:
status-desktop/VERSION
Line 1 in bdc7155
The text was updated successfully, but these errors were encountered: