-
Notifications
You must be signed in to change notification settings - Fork 14
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
Improve versioning of pgsphere #20
Comments
Dear All, I would like to agree on versioning of pgsphere:
What I'm concerned, now we increment only the third number (patch number) for those changes that include API changes. I guess we should increment minor number in this case. But if we decide to increment minor number on each API small change this number may become too big. One more question - should we increment minor or patch number each time when merging a new PR or introduce release plan? Another issue is an optional compilation of some parts like healpix. I do not exclude that some new optional compilation will appear in the near future. I think it should be the user responsibility how to compile. The version number should not reflect how pgsphere is compiled. I see the only issue - upgrade scripts. I guess we may implement some query function that returns the list of compiled features. It may help upgrade script to decide what to do. I appreciate any comments, thank you! |
It seems this issue is a duplicate of #9. Closing it. |
I didn't see these as duplicates, but I'm ok with combining them. I thought this issue was for releasing new versions at https://github.com/postgrespro/pgsphere/releases. I thought issue #9 was going back in the commit history of repository and tagging old releases. |
Well, I thought that it is a single complex problem with versions. Furthermore, there were some talks about producing release artifacts. It is why I decided to have the single issue for discussion. Anyway, I have no objections to reopen this Issue. |
Yes, I agree with that. |
Introduce releases
The text was updated successfully, but these errors were encountered: