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 components of the version to duckdb.hpp #10840

Merged
merged 6 commits into from
Feb 27, 2024

Conversation

ahuarte47
Copy link
Contributor

This simple PR adds the three components of the version to the duckdb.hpp.

It would be useful to detect breaking changes between releases, for example to define #ifdef preprocesor directives to write conditional includes or typedefs.

An example is the renaming of the PreservedError class from v0.9.x to ErrorData in the v0.10.x releases.

@github-actions github-actions bot marked this pull request as draft February 25, 2024 19:08
@ahuarte47 ahuarte47 marked this pull request as ready for review February 25, 2024 19:34
@github-actions github-actions bot marked this pull request as draft February 25, 2024 19:43
@ahuarte47 ahuarte47 marked this pull request as ready for review February 25, 2024 20:21
@github-actions github-actions bot marked this pull request as draft February 25, 2024 20:26
@ahuarte47 ahuarte47 marked this pull request as ready for review February 25, 2024 20:26
@Mytherin Mytherin merged commit 3345f75 into duckdb:main Feb 27, 2024
46 checks passed
@Mytherin
Copy link
Collaborator

Thanks! Makes sense

@ahuarte47
Copy link
Contributor Author

Thanks @Mytherin !!!

@ahuarte47 ahuarte47 deleted the main_add-version-parts branch February 27, 2024 13:20
github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Mar 15, 2024
Merge pull request duckdb/duckdb#10714 from pdet/header_default_true
Merge pull request duckdb/duckdb#10840 from ahuarte47/main_add-version-parts
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