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

Arrangement: small doc fixes #7660

Merged
merged 5 commits into from
Sep 7, 2023
Merged

Conversation

afabri
Copy link
Member

@afabri afabri commented Aug 22, 2023

Summary of Changes

Throw in % to avoid links.

Release Management

  • Affected package(s): Arrangements

@albert-github
Copy link
Contributor

I think good corrections , though I think there are more places in the CGAL software where these problems occur (probably also the word Cartesian a bit earlier in the same file.
I think a grep / grep -v might be useful here

  • check Cartesian (and also cartesian)
  • check orientation()
  • check Homogeneous (and also homogeneous)

@afabri
Copy link
Member Author

afabri commented Aug 22, 2023

No need for the lower case versions.

@afabri
Copy link
Member Author

afabri commented Aug 22, 2023

So far I did only fixes in Pkg/doc/Pkg

@albert-github
Copy link
Contributor

Regarding lower case conversions I wrote this as a reminder to check whether or not the case of the words is correct or not.

@afabri
Copy link
Member Author

afabri commented Aug 22, 2023

Regarding lower case conversions I wrote this as a reminder to check whether or not the case of the words is correct or not.

ok. In fact I changed several "Homogeneous" to "homogeneous".

@sloriot
Copy link
Member

sloriot commented Sep 6, 2023

Successfully tested in CGAL-6.0-Ic-56

@lrineau lrineau self-assigned this Sep 6, 2023
@lrineau lrineau added the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Sep 6, 2023
@lrineau lrineau merged commit 6ce9667 into CGAL:master Sep 7, 2023
@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 Sep 7, 2023
@lrineau lrineau deleted the Arrangement-typo-GF branch September 7, 2023 09:08
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.

4 participants