-
Notifications
You must be signed in to change notification settings - Fork 21
treehouses remote commands (fixes #829) #864
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
Conversation
LordJashin32
left a comment
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.
Interesting. This will be a slight pain to deal with once we implement _treehosues fully fix. I think we can only add more places where will we need to update once we add a new command lol.
|
I'm thinking of doing some sorta grep on examples. |
|
Missing some commands for services. |
Yeah. I think there's no way to avoid not doing it manually. I looked into auto completion and there's no way to just list everything u want. EDIT: woops accidentally closed it for a sec.Gonna send you the output of my tests run in 10 mins then u will have em all sorry for the wait. |
Yeah I used a script to get the list but to get the second and third level commands it would require like 4 or 5 nested for loops. There's probably a smarter way to do it. |
|
Test script commands Would probably add some new commands not in the script yet like: |
d2c1c83 to
cafe90a
Compare
|
rebased branch on new master |
|
Tested: |
|
Nice update. There are a few commands here and there that are missing / incorrect. Hopefully a one time thing. |
Fixes #829
Not sure if this is the approach @dogi was looking for.