-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
Keploy server version #216
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you and congratulations 🎉 for opening your very first pull request in keploy
I have read the CLA Document and I hereby sign the CLA |
…tartup Signed-off-by: Pankhudi Bhonsle <psbhonsle08@gmail.com>
Signed-off-by: Pankhudi Bhonsle <psbhonsle08@gmail.com>
ea7b32d
to
9d6c342
Compare
@@ -22,7 +22,9 @@ jobs: | |||
go-version: 1.17 | |||
|
|||
- name: Build | |||
run: go build -v ./... | |||
run: | | |||
git_hash=$(git rev-parse --short "$GITHUB_SHA") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PankhudiB as discussed in the issue thread. We should have both the commit-sha and the git-tag logged for the version. We mostly use git-tag for versions but adding commit-sha can. help identify unreleased builds.
@@ -7,7 +7,10 @@ import ( | |||
"go.keploy.io/server/server" | |||
) | |||
|
|||
// Version will be injected during go build with ldflag | |||
var Version = "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should also have a default value, maybe something like development
Related Issue
Closes: #206
Describe the changes you've made
Log the version during keploy server startup. Inject the version during go build with ldflags
Type of change
Checklist:
Local run output by passing dummy version: