Skip to content

style: ✏️ Cleanup: Remove todos #135

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 26, 2023
Merged

style: ✏️ Cleanup: Remove todos #135

merged 1 commit into from
Feb 26, 2023

Conversation

ithinkandicode
Copy link
Collaborator

Removes a few "todo" comments. They'd be better as issues if we still want to implement them, as that would help us keep track of them better.

Qubus0
Qubus0 approved these changes Feb 26, 2023
View reviewed changes
Copy link
Collaborator

@Qubus0 Qubus0 left a comment

Choose a reason for hiding this comment

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

Ay. Don't think we need them as issues. They are all covered in a different way or part of the UI which is on hold for now

@ithinkandicode ithinkandicode merged commit dcbc0e3 into GodotModding:development Feb 26, 2023
@ithinkandicode ithinkandicode added the refactor / cleanup Improves readability or maintainability label Feb 27, 2023
@ithinkandicode ithinkandicode deleted the cleanup-todos branch February 28, 2023 22:47
@ithinkandicode ithinkandicode changed the title Cleanup: Remove todos style: ✏️ Cleanup: Remove todos Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Qubus0 Qubus0 Qubus0 approved these changes

Assignees
No one assigned
Labels
refactor / cleanup Improves readability or maintainability
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ithinkandicode @Qubus0