-
Notifications
You must be signed in to change notification settings - Fork 5
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
Change "clear" to "reset" and add new commands #90
Change "clear" to "reset" and add new commands #90
Conversation
Codecov Report
@@ Coverage Diff @@
## master #90 +/- ##
============================================
- Coverage 73.98% 73.62% -0.37%
- Complexity 461 525 +64
============================================
Files 77 87 +10
Lines 1357 1596 +239
Branches 145 180 +35
============================================
+ Hits 1004 1175 +171
- Misses 312 368 +56
- Partials 41 53 +12
Continue to review full report at Codecov.
|
docs/UserGuide.md
Outdated
Assigns a professor to a module. | ||
|
||
Format: `assign INDEX m/MODULE_CODE [MORE MODULE_CODES]` | ||
* Assigns all the `MODULE_CODE` to the contact at the specified `INDEX`. All of the `MODULE_CODE` **must exist** in FaculType in the first place. |
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.
Hmm for this part is it better to say "Assigns the contact/instructor at the specified index to the module code"? 🤔
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 have made some changes. Can you check it again? thanks.
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!
Fixes #79