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

Rename addFillet files to addEdgeTreatment #4644

Merged
merged 3 commits into from
Dec 3, 2024
Merged

Conversation

max-mrgrsk
Copy link
Collaborator

Description:

Renamed addFillet.ts and addFillet.test.ts to addEdgeTreatment.ts and addEdgeTreatment.test.ts for consistency with the refactor. Updated all references across the codebase.

closes: #4642
Follow-up to PR #4593
This issue is part of the larger Fillet UI project: #2606

@max-mrgrsk max-mrgrsk linked an issue Dec 3, 2024 that may be closed by this pull request
Copy link

qa-wolf bot commented Dec 3, 2024

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

@max-mrgrsk max-mrgrsk self-assigned this Dec 3, 2024
Copy link

vercel bot commented Dec 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview Dec 3, 2024 8:53pm

@max-mrgrsk max-mrgrsk marked this pull request as ready for review December 3, 2024 21:08
@max-mrgrsk max-mrgrsk requested a review from Irev-Dev December 3, 2024 21:08
@Irev-Dev Irev-Dev merged commit 776b420 into main Dec 3, 2024
26 checks passed
@Irev-Dev Irev-Dev deleted the max-rename-addfilletts branch December 3, 2024 21:30
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.

rename addFillet.ts to addEdgeTreatment.ts
2 participants