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 #710

Merged
merged 26 commits into from
Mar 17, 2019
Merged

Refactor #710

merged 26 commits into from
Mar 17, 2019

Conversation

NeoTheThird
Copy link
Member

@NeoTheThird NeoTheThird commented Mar 15, 2019

These are the initial steps of our long expected refactoring! Testing is done on the interim release. I definitely want at least one entry in every row and one in every column before merging, but maybe we can already get started on code reviews :)

New features

Bugfixes

General improvements related to unreported or distributed issues

  • More intuitive loading bars
  • Improve logging by, like, a lot
  • Remove a lot of unused code
  • Improvements of code quality in some areas
  • Move stuff around
  • More useful event handling

These are the initial steps of our long expected refactoring.

- Include the command-line interface in the normal app (even though it
does not do anything yet)
- Tame the crossfiring commands, fixes #187 and possibly many other
issues
- Fixes stupid bug where we tried to execute [Object]
- More intuitive loading bars
- Improve logging by, like, a lot
- Remove a lot of unused code
- Improvements of code quality in some areas
- Move stuff around
- More useful event handling
- Finish adb-support snapd interface implementation, fixes #258
- Implement ubports-api-node-module
- Use new (a year old, holy shit time is flying!) API server, fixes #692
- Shut the child process up, fixes #216
- Fix macos paths
…dling and fixed a weird issue that caused the particle animation to only show up in debug mode
…-fallback, resolves #705. Add --debug and --verbose, resolves #704. Fixes #703 where native tools were used even when custom tools are specified. Improve readme documentation.
@NeoTheThird NeoTheThird self-assigned this Mar 15, 2019
Copy link
Member

@mariogrip mariogrip left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Good work!

@NeoTheThird
Copy link
Member Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment