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

Refactoring backlog #11

Open
3 of 16 tasks
saeub opened this issue Mar 3, 2021 · 0 comments
Open
3 of 16 tasks

Refactoring backlog #11

saeub opened this issue Mar 3, 2021 · 0 comments

Comments

@saeub
Copy link
Owner

saeub commented Mar 3, 2021

  • Consistent use/omission of const in arguments
  • Prefer required keyword arguments over many positional arguments in constructors
  • Doc comments
  • More sane type casting in JSON deserialization
  • Rethink testing without assets (e.g. reaction time task)
  • Increase test coverage
  • Null safety
  • Replace QR package (ML Kit?)
  • Remove stopwatch?
  • Fix stretched loading animation on AboutListTile
  • Somehow prevent passing mutable objects as data when logging (logged values must be frozen); having to remember to put .toList() every time is not very nice (see b9594ad)
  • Organize imports (relative within package, sorted)
  • Use ApiError.retriable to consistently show retriable/unretriable error messages
  • Use final instead of var whenever possible (good reason)
  • Rename subtask to trial
  • Put default/initial values for stateful widgets in field definition or constructor, use initState only if access context is required
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

No branches or pull requests

1 participant