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

fix: remove build action workflow #53

Merged
merged 2 commits into from
Jan 9, 2022
Merged

fix: remove build action workflow #53

merged 2 commits into from
Jan 9, 2022

Conversation

StewartJingga
Copy link
Contributor

Fix github's build action workflow due to code changes made to cater new releaseaction workflow.

@StewartJingga StewartJingga requested a review from mabdh January 9, 2022 11:08
@StewartJingga StewartJingga self-assigned this Jan 9, 2022
on:
push:
tags:
- "v*.*.*"
branches:
Copy link
Member

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?

Copy link
Contributor Author

@StewartJingga StewartJingga Jan 9, 2022

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.

Copy link
Member

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?

Copy link
Contributor Author

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?

Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ravisuhag removed 🙏

@StewartJingga StewartJingga changed the title fix: git build action workflow fix: remove build action workflow Jan 9, 2022
@ravisuhag ravisuhag merged commit 4bb4d5b into main Jan 9, 2022
@ravisuhag ravisuhag deleted the fix-build-workflow branch January 9, 2022 13:22
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.

2 participants