Skip to content

chore: bump foxtrot to 87c282b - #1463

Merged
akhilles merged 1 commit into
mainfrom
akhil/foxtrot-polyloop
Oct 10, 2026
Merged

akhilles merged 1 commit into
mainfrom
akhil/foxtrot-polyloop

Conversation

@akhilles

@akhilles akhilles commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

This change bumps foxtrot to pick up diodeinc/foxtrot#25 (ENG-3787). Faceted STEP models with POLY_LOOP faces, such as the Diode registry's iges2facetstep output, now tessellate instead of being dropped from GLB export. On a 300-board corpus sample, only the board that contains such models changes.


Devin Review

Picks up foxtrot #25 (ENG-3787): faceted STEP models with POLY_LOOP faces,
such as the Diode registry's iges2facetstep output, now tessellate.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not approved: this foxtrot bump changes GLB/STEP tessellation behavior, which is a functional user-facing change rather than a small fixup, so human review is needed. Cursor Security Agent completed with no findings; Cursor Bugbot was not present. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@akhilles
akhilles merged commit db35f77 into main Oct 10, 2026
14 checks passed
@akhilles
akhilles deleted the akhil/foxtrot-polyloop branch October 10, 2026 19:12
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