Skip to content
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

Show build number in startup log and version check command #4336

Merged
merged 18 commits into from
Jun 22, 2024

Conversation

onebeastchris
Copy link
Member

@onebeastchris onebeastchris commented Dec 13, 2023

Changes made:

TODO's:

  • yeet md5 hash from dump? - the downloads api does not provide it

Would fix #1748

Don't check build number if running a dev build
Show dev build warning in console
@onebeastchris onebeastchris added PR: Feature When a PR implements a new feature PR: Cosmetic When a PR adds something that is purely cosmetic, like changing the README labels Dec 13, 2023
@onebeastchris onebeastchris added the PR: Needs review Indicates that a PR is functional and review-ready. label Dec 16, 2023
onebeastchris and others added 5 commits January 2, 2024 17:42
Changed GeyserImpl.VERSION to consist of build number (e.g 420) + git info instead of geyser version (2.2.0-SNAPSHOT) + git info
…dev to reflect build version

- make isDev a variable in the GitInfo class
@onebeastchris onebeastchris requested a review from Tim203 January 22, 2024 09:01
@onebeastchris onebeastchris added PR: On hold When a PR is on hold like if it requires a dependency to be updated first and removed PR: Needs review Indicates that a PR is functional and review-ready. labels Jan 22, 2024
@Kas-tle
Copy link
Member

Kas-tle commented Apr 20, 2024

Must be adapted as build number is no longer pulled directly from GH actions and instead uses the env variable BUILD_NUMBER.
See https://github.com/GeyserMC/Geyser/pull/4583/files#diff-a6fced4b4e4a99784a2b92028b1adbc8c0ca28ff54e264b0e73727b3d4a460cf.

…d-number

# Conflicts:
#	core/build.gradle.kts
@onebeastchris onebeastchris changed the title Show github actions build number in startup log and version check command Show build number in startup log and version check command Jun 20, 2024
@onebeastchris onebeastchris removed the PR: On hold When a PR is on hold like if it requires a dependency to be updated first label Jun 22, 2024
@onebeastchris onebeastchris merged commit 55e90b6 into GeyserMC:master Jun 22, 2024
2 checks passed
@onebeastchris onebeastchris deleted the build-number branch June 22, 2024 18:33
XingLingQAQ pushed a commit to XingLingQAQ/Geyser that referenced this pull request Sep 25, 2024
…#4336)

* Version check command/startup log shows build number
* Add development build warning
* Fix `/geyser version` command
* Hack around outdated language module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Cosmetic When a PR adds something that is purely cosmetic, like changing the README PR: Feature When a PR implements a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include Build Number in Start logs and /geyser version
5 participants