Skip to content

SK-2315 add version as prefix to logs #251

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.

Sign up for GitHub

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

Jump to bottom
Merged
skyflow-shravan merged 1 commit into from
Oct 3, 2025
Merged

SK-2315 add version as prefix to logs #251

skyflow-shravan merged 1 commit into from
Oct 3, 2025

Conversation

@skyflow-shravan
Copy link
Contributor

@skyflow-shravan skyflow-shravan commented Oct 3, 2025

Why

  • To get the SDK version as prefix for the logs
  • The version was dynamically hard coded before in constants file
  • Now we get it from the project metadata

Goal

  • The logs should have the Skyflow Java version as prefix

@skyflow-shravan skyflow-shravan changed the base branch from v3 to release/25.10.1 October 3, 2025 11:27
@skyflow-shravan skyflow-shravan changed the base branch from release/25.10.1 to v3-release/25.10.1 October 3, 2025 11:31
@skyflow-shravan skyflow-shravan merged commit 29215f7 into v3-release/25.10.1 Oct 3, 2025
4 checks passed
skyflow-shravan added a commit that referenced this pull request Oct 3, 2025
@skyflow-shravan
SK-2315 add version as prefix to logs (#252)
7c5c85e 
* SK-2315 add version as prefix to logs (#251)

* [AUTOMATED] Private Release 3.0.0-beta.6-dev-29215f7

* SK-2315 update the samples

* [AUTOMATED] Private Release 3.0.0-beta.6-dev-4dec765

---------

Co-authored-by: skyflow-shravan <skyflow-shravan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@skyflow-bharti skyflow-bharti skyflow-bharti approved these changes

@skyflow-vivek skyflow-vivek skyflow-vivek approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@skyflow-shravan @skyflow-bharti @skyflow-vivek