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.

Sign up for GitHub

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

Jump to bottom

Polyline_simplification: fix documentation #8119

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Polyline_simplification: fix documentation #8119

merged 1 commit into from
Apr 10, 2024

Conversation

afabri
Copy link
Member

@afabri afabri commented Apr 4, 2024

Summary of Changes

smaller -> greater in documentation

Release Management

@afabri afabri added this to the 5.6.2 milestone Apr 4, 2024
@afabri afabri requested a review from MaelRL April 4, 2024 06:59
@afabri afabri self-assigned this Apr 4, 2024
@lrineau lrineau linked an issue Apr 5, 2024 that may be closed by this pull request
Documentation in CGAL::Polyline_simplification_2::Stop_above_cost_threshold Class #8114
Closed
@afabri
Copy link
Member Author

afabri commented Apr 5, 2024

Successfully tested in Ic-211

@afabri afabri added the Tested label Apr 5, 2024
@lrineau lrineau added trivial rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' labels Apr 5, 2024
@sloriot
Copy link
Member

sloriot commented Apr 10, 2024

Successfully tested in CGAL-6.0-Ic-21[56]

@lrineau lrineau merged commit 3575144 into CGAL:5.6.x-branch Apr 10, 2024
7 checks passed
lrineau added a commit that referenced this pull request Apr 10, 2024
@lrineau
Merge pull request #8119 from afabri/Polyline_simplification-doc_bug-GF
d7bf7dd 
Polyline_simplification: fix documentation
@lrineau lrineau removed the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Apr 10, 2024
@lrineau lrineau deleted the Polyline_simplification-doc_bug-GF branch April 10, 2024 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaelRL MaelRL Awaiting requested review from MaelRL

Assignees

@afabri afabri

Labels
Doc bug Merged_in_5.6.2 Merged_in_6.0 Pkg::Polyline_simplification_2 Tested trivial
Projects
None yet
Milestone
5.6.2
Development

Successfully merging this pull request may close these issues.

Documentation in CGAL::Polyline_simplification_2::Stop_above_cost_threshold Class
3 participants
@afabri @sloriot @lrineau