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

➕ Add bump-my-version to dev dependencies #529

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

stevenbal
Copy link
Collaborator

Fixes maykinmedia/open-api-framework#107 partially

Changes

  • Add bump-my-version to dev dependencies

@stevenbal stevenbal force-pushed the feature/bump-my-version branch from 1691b0f to 7eeecd5 Compare February 18, 2025 10:26
search = "version": "{current_version}",
replace = "version": "{new_version}",

[bumpversion:file:src/objects/conf/api.py]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@annashamray I think this was incorrectly part of bumpversion.cfg, it includes the API version which I think are separate from the version of the software, right?

Copy link
Collaborator

Choose a reason for hiding this comment

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

It's not an easy question actually. Maybe we need to discuss it with @joeribekker
When we supported v1 and v2 of API at the same time we had Objects software version == Objects API v2 version

Now we dropped v1 of the API and released v3 of software and they started to deviate from each other.
I personally don't like it. Objects API is mostly an Restful API with a little bit of admin interface, so it doesn't make a lot of sense to have separate versions. Moreover unlike OZ we don't have any standard to adhere to.
I think having one version would be less confusing and more convenient for both developers and clients

But it's up to discussion

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Let's discuss it when Joeri is back indeed

@stevenbal stevenbal merged commit 384e9cc into master Feb 24, 2025
13 checks passed
@stevenbal stevenbal deleted the feature/bump-my-version branch February 24, 2025 12: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.

Add release issue template for all components
2 participants