-
-
Notifications
You must be signed in to change notification settings - Fork 219
attempted to improve command organization #1236
Conversation
Jesus this is going to take a while to review, I was not expecting someone besides me to actually make this. |
Btw, two more things:
|
Definitely not. It's a file used by macOS that stores custom attributes of its containing folder, such as the position of icons etc. |
Gosh darn macOS adding hidden files. I'll remove them from the PR, and add .DS_Store to the gitignore |
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.
LGTM
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 just noticed the target branch is master, not dev. However, changing it now will probably cause lots of conflicts. On the other hand, if we end up merging this into master instead, it will cause conflicts when merging dev -> master.
It's on his fork, he can cancel the PR and create a new one targeting dev and them resolving the conflits. |
We probably gonna use your PR as a base for categories on @switchblade-next, anyway, thanks for opening it. |
Reorganized the commands as best as I could discern, also made sure each command has the correct
category
property. Addresses issue #1189