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

Tag versions to allow stable point references #145

Closed
jcourt562 opened this issue May 14, 2021 · 10 comments
Closed

Tag versions to allow stable point references #145

jcourt562 opened this issue May 14, 2021 · 10 comments
Assignees

Comments

@jcourt562
Copy link
Contributor

VON network is an extremely useful tool for developers before they are ready to brave the public test ledgers on Sovrin, Indicio etc.

It would be great If we could just tag the current version so that future development, that might break the HEAD does not effect people pulling the repo. I know they could just pull a specific commit ref but a tag makes it more obvious which one to use.

@swcurran
Copy link
Contributor

@WadeBarnes -- what do you think? Presumably the locked version would have to lock down dependencies, at least to some extent. For example, using a specific version of von-image, which in turn would use a specific version of Indy.

@mgmgwi
Copy link
Contributor

mgmgwi commented May 21, 2021

@WadeBarnes: Would you consider this PR IDunion/pull/1 ? If it's ok, I will create a new PR here. The idea is to provide
a) a ready to use image (without having to build it), and
b) provide labels for specific versions

@WadeBarnes
Copy link
Member

@WadeBarnes: Would you consider this PR IDunion/pull/1 ? If it's ok, I will create a new PR here. The idea is to provide
a) a ready to use image (without having to build it), and
b) provide labels for specific versions

@mgmgwi, Absolutely. I've only had a quick look, but it looks good. We may end up making a repository over here (https://hub.docker.com/u/bcgovimages) for the built image, but feel free to submit the PR and I'll start having a closer look.

@WadeBarnes
Copy link
Member

WadeBarnes commented Jun 8, 2021

@mgmgwi, Thanks for PR #146, I've merged it and made a few updates to ensure we're not proliferating von-network images across all the forks. I also added a latest tag. Let me know what you think. I also renamed the image to von-network-base to be consistent with what the manage script and docker-compose file are expecting.

@WadeBarnes
Copy link
Member

@jcourt562, @mgmgwi, @swcurran, Do any of you have an opinion on the version number for the initial release tag?

@mgmgwi
Copy link
Contributor

mgmgwi commented Jun 8, 2021

@mgmgwi, Thanks for PR #146, I've merged it and made a few updates to ensure we're not proliferating von-network images across all the forks. I also added a latest tag. Let me know what you think. I also renamed the image to von-network-base to be consistent with what the manage script and docker-compose file are expecting.

That's great. Thank you very much!

Do any of you have an opinion on the version number for the initial release tag?

Not really, maybe you align this to bcgovimages/von-image:node-1.12-3 🤷‍♂️

@WadeBarnes
Copy link
Member

Not really, maybe you align this to bcgovimages/von-image:node-1.12-3 🤷‍♂️

That was my initial thought too, however we have made other updates to von-network itself in the past (and likely to again in the future) that warrant version bumps. So it's likely best to give von-network it's own version, I'm just wondering where we should start; <1.0.0. or >=1.0.0

@mgmgwi
Copy link
Contributor

mgmgwi commented Jun 8, 2021

Not really, maybe you align this to bcgovimages/von-image:node-1.12-3 man_shrugging

That was my initial thought too, however we have made other updates to von-network itself in the past (and likely to again in the future) that warrant version bumps. So it's likely best to give von-network it's own version, I'm just wondering where we should start; <1.0.0. or >=1.0.0

According to semver.org you probably should start with >=1.0.0.

@jcourt562
Copy link
Contributor Author

Happy with any version but since it really is in effectively public use already I would suggest just starting with 1.0.0

@WadeBarnes
Copy link
Member

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

No branches or pull requests

4 participants