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

change module version to v2 #132

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

richardbremner
Copy link
Collaborator

Review Type Requested (choose one):

  • [ x ] Glance - superficial check (from domain experts)
  • Logic - thorough check (from everybody doing review)

Summary

Upping the version to v2 and we'll use manual tagging to identify releases (for now)

Task/Issue reference

Closes: add_link_here

Details (optional)

We will leave the current circleci short sha tag versioning in place for now.

When we release we'll just create a tag with the next version number, ie.

git tag v2.0.x
git push origin v2.0.x

go install github.com/Lilypad-Tech/lilypad@latest will automatically pull down the highest semantic version

@richardbremner
Copy link
Collaborator Author

fixing up a whole bunch of "/v2" needing added through the codebase

Copy link
Contributor

@AquiGorka AquiGorka left a comment

Choose a reason for hiding this comment

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

@richardbremner approving to unblock, but I am not sure moving all the packages inside a v2 namespace is the best thing to do, does it bring any benefit that we could put in place without such a big move?

@AquiGorka
Copy link
Contributor

@richardbremner is this PR still relevant?

@richardbremner
Copy link
Collaborator Author

@AquiGorka i think it's still relevant, because it fixes the go install, which is otherwise still broken

@AquiGorka
Copy link
Contributor

If that's the case, then we should rebase, resolve the conflicts and merge.

@AquiGorka AquiGorka removed the request for review from Zorlin June 24, 2024 19:33
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.

None yet

2 participants