-
Notifications
You must be signed in to change notification settings - Fork 11
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
Various docs articles #5
Conversation
"Commands" implies its a lsit of commands, but it seems to focus more on code rather than the commands.
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.
While the suggested 'ls' and 'scn' commands are great, I don't want to indirectly promise anything by having it mentioned in the main branch docs.
You could open a new issue suggesting these commands and attaching a pull request with the ls
and scn
command docs and later we could add the commands themself.
Right now I want to focus on usability and polish, but if you implement the commands, I would be very happy.
I don't currently have much time because of finals, but I will soon return to work on GDShell and then I will make some new features
Anyway I really like your work and I very much appreciate it. The 'index' and 'man' pages are really good and if we document all the commands like this, that would be really awesome!
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
Co-authored-by: Jakub Janšta <[email protected]>
I've made the chnages you've suggested, good to merge? |
Seems good to me. Thanks! ❤️ |
No description provided.