Skip to content

Austin/legacy csg#1132

Draft
austin-j-knutson wants to merge 2 commits intomainfrom
austin/legacy-csg
Draft

Austin/legacy csg#1132
austin-j-knutson wants to merge 2 commits intomainfrom
austin/legacy-csg

Conversation

@austin-j-knutson
Copy link

@austin-j-knutson austin-j-knutson commented Mar 13, 2026

@austin-j-knutson austin-j-knutson marked this pull request as draft March 13, 2026 22:34
#[serde(default)]
#[builder(default)]
pub separate_bodies: bool,
// If true, use the legacy CSG algorithm.
Copy link
Contributor

Choose a reason for hiding this comment

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

Documentation comment uses // instead of ///. This is inconsistent with all other occurrences of the same field in this file (lines 1258, 1296, 2215, 2270, 2300) which use ///.

This means the documentation for this field will not be generated properly for this struct, making the API inconsistent. Should be:

/// If true, use the legacy CSG algorithm.
Suggested change
// If true, use the legacy CSG algorithm.
/// If true, use the legacy CSG algorithm.

Spotted by Graphite

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

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.

1 participant