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

Add OS version to gammaray status message #1588

Merged
merged 1 commit into from
Jan 18, 2025

Conversation

knoellle
Copy link
Contributor

Why? What?

Having juggled a bunch of branches with different OS versions recently, I got tired of guessing which OS version gammaray was actually uploading, so I added it to the status messages.

Should™️ be conflict-free with #1486

Fixes #

ToDo / Known Issues

Ideas for Next Iterations (Not This PR)

Also print the currently installed version on each robot, i.e. v10.5.42 -> v10.6.1

How to Test

Gammaray a NAO and watch the status messages.

@knoellle knoellle added the tools:Tooling Related to pepsi et.al. label Jan 18, 2025
@knoellle knoellle requested a review from schmidma January 18, 2025 14:50
@knoellle knoellle enabled auto-merge January 18, 2025 14:50
Copy link
Contributor

@oleflb oleflb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice

@knoellle knoellle added this pull request to the merge queue Jan 18, 2025
Merged via the queue into HULKs:main with commit 447c887 Jan 18, 2025
25 checks passed
@knoellle knoellle deleted the pepsi-gammaray-status-messages branch January 18, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tools:Tooling Related to pepsi et.al.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants