Skip to content
This repository has been archived by the owner on Sep 25, 2024. It is now read-only.

Split Long Methods #15

Open
Trip-kun opened this issue Nov 3, 2023 · 0 comments · May be fixed by #48
Open

Split Long Methods #15

Trip-kun opened this issue Nov 3, 2023 · 0 comments · May be fixed by #48
Assignees
@Trip-kun
Labels
good first issue Good for newcomers Improvement A feature works, but it could work better

Comments

@Trip-kun
Copy link

Trip-kun commented Nov 3, 2023

Certain Methods like Heartbeat run() are very long and should be split into multiple functions to better increase readability. Other long methods can be discussed here.
@Trip-kun Trip-kun added good first issue Good for newcomers Improvement A feature works, but it could work better labels Nov 3, 2023
@Trip-kun Trip-kun self-assigned this Nov 3, 2023
@Trip-kun Trip-kun mentioned this issue Nov 3, 2023
Merged
@Trip-kun Trip-kun linked a pull request Jan 14, 2024 that will close this issue
clean(managers): Split run method of DataManager into saveAndRemoveAl… #48
Draft
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Trip-kun Trip-kun

Labels
good first issue Good for newcomers Improvement A feature works, but it could work better
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

clean(managers): Split run method of DataManager into saveAndRemoveAl… frc5183/Oatmeal
1 participant
@Trip-kun