Skip to content

Set main version to 2.13.99#5818

Merged
chrchr-github merged 5 commits into
mainfrom
chrchr-github-patch-1
Jan 4, 2024
Merged

Set main version to 2.13.99#5818
chrchr-github merged 5 commits into
mainfrom
chrchr-github-patch-1

Conversation

@chrchr-github

Copy link
Copy Markdown
Collaborator

No description provided.

@danmar

danmar commented Jan 3, 2024

Copy link
Copy Markdown
Collaborator

hmm .. there are more versions to change. See c23521a

@danmar

danmar commented Jan 3, 2024

Copy link
Copy Markdown
Collaborator

btw there is a strange CPPCHECK_DEVMINOR_VERSION handling in version.h it would be good to tweak this handling.

@chrchr-github

Copy link
Copy Markdown
Collaborator Author

hmm .. there are more versions to change. See c23521a

Can't we merge that commit into main then?

@danmar

danmar commented Jan 3, 2024

Copy link
Copy Markdown
Collaborator

Can't we merge that commit into main then?

That does not help you have to also tweak all the versions from release-version to debug-version then..

@danmar

danmar commented Jan 3, 2024

Copy link
Copy Markdown
Collaborator

I would suggest that we rename CPPCHECK_DEVMINOR_VERSION to CPPCHECK_BUGFIX_VERSION. It can always be "99" for dev builds. When I make a release branch I think it would make sense that I increment the MINOR_VERSION and zero the BUGFIX_VERSION.

@chrchr-github chrchr-github changed the title Update productInfo.wxi Set main version to 2.13.99 Jan 3, 2024
Comment thread lib/version.h Outdated
Comment thread lib/version.h

@danmar danmar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@chrchr-github chrchr-github merged commit efa8a08 into main Jan 4, 2024
@chrchr-github chrchr-github deleted the chrchr-github-patch-1 branch January 4, 2024 21:00
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.

2 participants