-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix: remove build action workflow #53
Conversation
.github/workflows/build.yml
Outdated
on: | ||
push: | ||
tags: | ||
- "v*.*.*" | ||
branches: |
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.
@StewartJingga Any reason we want to release docker image on every commit?
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.
Was just letting people to get the latest update/fix on main branch only.
Right now in our current setup latest
tag in docker means latest main branch. unless we want to change it to mean latest tag.
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.
I think we should point the latest docker tag to the latest stable release and we can push a nightly build with a different tag for the main branch build. WDYT?
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.
sounds good to me. after reading a few articles about latest tag, it seems like it is to be avoided if possible.
latest is basically just another tag that is being added if there is no tag presents.
So maybe we could just remove this build
action unless we want a special tag for this main branch called main
perhaps? wdyt?
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.
For now, I think we can remove it. If later we decide to publish the nightly build we can include it.
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.
@ravisuhag removed 🙏
Fix github's
build
action workflow due to code changes made to cater newrelease
action workflow.