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

refactor: convert functions to methods #27

Open
7 tasks
GixoXYZ opened this issue Jan 1, 2024 · 0 comments
Open
7 tasks

refactor: convert functions to methods #27

GixoXYZ opened this issue Jan 1, 2024 · 0 comments
Assignees
Labels
effort: high This issue takes a large amount of work module: other This issue is related to other parts of the add-on priority: low This issue has a low priority refactor Code improvements
Milestone

Comments

@GixoXYZ
Copy link
Owner

GixoXYZ commented Jan 1, 2024

Description and Problem
Functions are scattered.

Proposed Changes
Convert functions to class methods where it makes sense.

Benefits
Cleaner, more unified code.

Tests

  • Check the code under normal conditions.
  • Check the code with problematic internet connection, unusual scenarios and repeatedly.
  • Check the results and reportings to make sure they work as intended.

Checklist

  • Proposed changes have been reviewed and approved.
  • Code has been reviewed and has passed the tests.
  • Documentation has been updated (if necessary).
  • Code follows coding standards.
@GixoXYZ GixoXYZ added module: other This issue is related to other parts of the add-on priority: low This issue has a low priority effort: medium This issue takes a medium amount of work refactor Code improvements labels Jan 1, 2024
@GixoXYZ GixoXYZ added this to the 2.3.0 milestone Jan 1, 2024
@GixoXYZ GixoXYZ self-assigned this Jan 1, 2024
@GixoXYZ GixoXYZ added this to GBHTool Jan 1, 2024
@GixoXYZ GixoXYZ moved this to Backlog in GBHTool Jan 1, 2024
@GixoXYZ GixoXYZ changed the title refactor(update checker): convert functions to methods refactor: convert functions to methods Jan 2, 2024
@GixoXYZ GixoXYZ added effort: high This issue takes a large amount of work and removed effort: medium This issue takes a medium amount of work labels Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: high This issue takes a large amount of work module: other This issue is related to other parts of the add-on priority: low This issue has a low priority refactor Code improvements
Projects
Status: Backlog
Development

No branches or pull requests

1 participant