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

chore: clean up generalized_FT #69

Merged
merged 7 commits into from
Feb 2, 2024
Merged

Conversation

cpaniaguam
Copy link
Member

Reorganize imports, remove unused names, simple refactorings

Reorganize imports, remove unused names, simple refactorings
@cpaniaguam cpaniaguam requested a review from kmilo9999 January 18, 2024 15:41
@cpaniaguam cpaniaguam mentioned this pull request Jan 18, 2024
Base automatically changed from fix-step2 to feat-step5-intg January 18, 2024 19:04
Base automatically changed from feat-step5-intg to main January 29, 2024 14:27
kmilo9999
kmilo9999 previously approved these changes Feb 2, 2024
@kmilo9999
Copy link
Collaborator

@cpaniaguam This one is ready to go.

fix: remove duplicate import and reorganize
@cpaniaguam cpaniaguam requested a review from kmilo9999 February 2, 2024 20:47
Copy link
Collaborator

@kmilo9999 kmilo9999 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@cpaniaguam cpaniaguam merged commit 5c60a90 into main Feb 2, 2024
1 check passed
@cpaniaguam cpaniaguam deleted the fix-step2-update-generalized_FT branch February 2, 2024 21:21
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.

2 participants