Skip to content

Conversation

@wawanbreton
Copy link
Contributor

@wawanbreton wawanbreton commented Dec 11, 2025

This PR removes the settings associated to:

  • Extra infill lines to support skin
  • Skin edge support layer
  • Bridging over infill

And adds the new settings for the "Infill skin support" that replaces them.

Requires Ultimaker/CuraEngine#2269

CURA-12361

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/4)

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (2/4)

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (3/4)

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (4/4)

@wawanbreton wawanbreton marked this pull request as ready for review December 11, 2025 15:51
@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit c63e0e9. ± Comparison against base commit b4eb120.

♻️ This comment has been updated with latest results.

Base automatically changed from CURA-12833_improve-bridge-lines-direction to main December 17, 2025 13:30
@HellAholic HellAholic merged commit 0c6d7d6 into main Dec 19, 2025
2 of 3 checks passed
@HellAholic HellAholic deleted the CURA-12361_add-skin-support branch December 19, 2025 14:18
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.

3 participants