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

Provide a numeric version number to compare against. #1054

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

hzeller
Copy link
Contributor

@hzeller hzeller commented Nov 17, 2024

Including simple example in version.h.in and checking in the test-{cmake,pkgconfig}.sh scripts.

Including simple example in version.h.in and checking
in the test-{cmake,pkgconfig}.sh scripts.
@hzeller
Copy link
Contributor Author

hzeller commented Nov 17, 2024

CC: @kintel @pca006132
Something like this ?

Copy link

codecov bot commented Nov 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.69%. Comparing base (d437097) to head (278d2b6).
Report is 157 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1054      +/-   ##
==========================================
- Coverage   91.84%   84.69%   -7.15%     
==========================================
  Files          37       62      +25     
  Lines        4976     9685    +4709     
  Branches        0     1050    +1050     
==========================================
+ Hits         4570     8203    +3633     
- Misses        406     1482    +1076     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hzeller
Copy link
Contributor Author

hzeller commented Nov 17, 2024

or maybe two macros, MANIFOLD_VERSION_NUMERIC which contains 2005001 and a MANIFOLD_VERSION_STRING which contains "2.5.1" ?

@hzeller hzeller mentioned this pull request Nov 17, 2024
Copy link
Owner

@elalish elalish left a comment

Choose a reason for hiding this comment

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

LGTM - anyone else have opinions, or anything we should hold v3.0 for?

@kintel
Copy link
Contributor

kintel commented Nov 18, 2024

@elalish Perhaps figure out the version.h in WASM mode: openscad/openscad#5434

@pca006132
Copy link
Collaborator

Ah I know that one, will open a PR

@elalish elalish merged commit 182adae into elalish:master Nov 18, 2024
23 checks passed
@elalish elalish mentioned this pull request Nov 18, 2024
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.

4 participants