-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
I thought about this feature, but I avoided it because I thought it might affect performance, but it seems to be important and if we make a simple format for the config file we can reduce the impact on performance |
## TODO | ||
|
||
- [ ] Add Tests | ||
- [ ] Add Formatter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean by formatter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean code formatter like prettier
It would be nice if you added a pre-commit hook that takes care of formatting and testing before commiting/pushing to the master branch.
…() and make it return Object[] instead of String
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
I think the code is much cleaner now and easier to read
please add a config file parser that parse config file which is defined by the user, the user can specify custom icons, colors and settings.
I forgot to add that in the TODO section.