-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement dynamic versioning for 2 or 3 MeltanoLabs
-managed connectors.
#22
Comments
MeltanoLabs
-managed connectors.MeltanoLabs
-managed connectors.
@aaronsteers with dynamic versioning in place, would users be installing from a git tag instead of the default branch? |
@edgarrmondragon If they installed from a branch instead of a tag, depending on how we configure the tool, they'd either get a dirty version (e.g. |
@WillDaSilva ah, that's nice. So if I understand the use case correctly, once a user publishes a new release & tag from the GitHub UI a workflow is triggered with the following steps:
This seems like a good approach, and we could probably make it even easier for each repo by wrapping the workflow described above in a reusable action. cc @aaronsteers |
@edgarrmondragon Yup. It wouldn't surprise me if such a workflow was already available on the marketplace as an action. It has the benefit of running the workflow for the release/tag itself, so there's no risk of drift between the commit that updates the version, and the commit that gets built into the published wheel/images. Note that this approach only works if building directly git already works, so it wouldn't, for instance, work for Meltano because we have to build the UI prior to building the wheel. |
@kgpayne - It occurs to me that |
MeltanoLabs
-managed connectors.MeltanoLabs
-managed connectors.
MeltanoLabs
-managed connectors.MeltanoLabs
-managed connectors.
Certainly could try it, very low risk no one's using this right now! :D |
Closing as complete. For example implementations, see
Fantastic work, @kgpayne ! 🚀 |
One challenge in maintaining multiple connectors is the tension between:
main
branch.This approach would eliminate the second requirement, so we could in theory release new versions without needing to commit a version bump change back to the repo:
From the readme:
The text was updated successfully, but these errors were encountered: