Skip to content

beam-init: add version endpoint - #132

Open
rcanderson23 wants to merge 1 commit into
mainfrom
carson/beam-init-version
Open

beam-init: add version endpoint#132
rcanderson23 wants to merge 1 commit into
mainfrom
carson/beam-init-version

Conversation

@rcanderson23

Copy link
Copy Markdown
Contributor

Adds version endpoint to beam-init as well as command to beamctl.

@rcanderson23
rcanderson23 force-pushed the carson/beam-init-version branch 5 times, most recently from bbe0df1 to e97f0c5 Compare August 11, 2026 18:35
@rcanderson23
rcanderson23 marked this pull request as ready for review August 11, 2026 18:36
@rcanderson23
rcanderson23 requested review from bjorn3, boxofrad, folkertdev and sclevine and removed request for bjorn3 August 11, 2026 18:37
Adds version endpoint to beam-init as well as command to beamctl.
@rcanderson23
rcanderson23 force-pushed the carson/beam-init-version branch from e97f0c5 to 62dfc8d Compare August 11, 2026 18:40
Comment thread beam-init/src/main.rs
pub(crate) const GIT_SHA: &str = match option_env!("GIT_SHA") {
Some(sha) => sha,
None => "unknown",
};

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe this could be put in beam-init/src/lib.rs to share it between beam-init and beamctl?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I originally did that but moved to individual bins since it required pub visibility

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think it would be a problem to make these pub in the beam-init library.

@bjorn3 bjorn3 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

With or without the consts moved to lib.rs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants