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 CGAL/version_checker.h #7528

Merged
merged 4 commits into from
Aug 14, 2023

Conversation

janetournois
Copy link
Member

Summary of Changes

Following discussion #7525 I added the header version_enforcer.h

In the general case, it has nothing to check. If the user includes his own fork of version_enforcer.h with a given number of CGAL version, then the check happens.

Release Management

  • Affected package(s): Installation

@janetournois janetournois added this to the 5.6 milestone Jun 15, 2023
@janetournois janetournois force-pushed the CGAL-add_version_enforcer-jtournois branch from c22eb19 to 522183b Compare June 15, 2023 14:17
@sloriot sloriot added the Not yet approved The feature or pull-request has not yet been approved. label Jun 15, 2023
@lrineau lrineau modified the milestones: 5.6, 5.6-beta2 Jun 22, 2023
the 3 major/minor/patch are needed to compare integrals

if they are defined, the version check is done and warning or error are displayed
@janetournois janetournois changed the title Add CGAL/version_enforcer.h Add CGAL/version_checker.h Jun 22, 2023
@sloriot sloriot added Under Testing Batch_2 Second Batch of PRs under testing Batch_1 First Batch of PRs under testing and removed Under Testing Batch_2 Second Batch of PRs under testing labels Jun 28, 2023
@sloriot
Copy link
Member

sloriot commented Jul 12, 2023

Successfully tested in CGAL-6.0-Ic-20

@sloriot sloriot added the Tested label Jul 12, 2023
@sloriot sloriot removed the Batch_1 First Batch of PRs under testing label Jul 12, 2023
@lrineau lrineau modified the milestones: 5.6-beta2, 5.6 Jul 20, 2023
@github-actions github-actions bot removed the Tested label Jul 24, 2023
@github-actions
Copy link

This pull-request was previously marked with the label Tested, but has been modified with new commits. That label has been removed.

@lrineau lrineau removed the Not yet approved The feature or pull-request has not yet been approved. label Jul 26, 2023
@lrineau lrineau self-assigned this Jul 26, 2023
@lrineau lrineau added Not yet approved The feature or pull-request has not yet been approved. rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' and removed rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' labels Jul 26, 2023
@lrineau
Copy link
Member

lrineau commented Jul 26, 2023

This PR has the milestone CGAL-5.6 but its base branch is master (for CGAL-6.0). Let me know what I should do.

@janetournois
Copy link
Member Author

If there are more things to be merged to 5.6, it can be merged there. Otherwise I can wait for the next release, no worries

@lrineau lrineau modified the milestones: 5.6, 5.6.1 Jul 28, 2023
@sloriot
Copy link
Member

sloriot commented Aug 4, 2023

Successfully tested in CGAL-6.0-Ic-33

@sloriot sloriot removed the Not yet approved The feature or pull-request has not yet been approved. label Aug 14, 2023
@sloriot sloriot assigned sloriot and unassigned lrineau Aug 14, 2023
@sloriot sloriot merged commit 2f3588b into CGAL:master Aug 14, 2023
@sloriot sloriot deleted the CGAL-add_version_enforcer-jtournois branch August 14, 2023 09:51
@lrineau lrineau modified the milestones: 5.6.1, 6.0-beta Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants