-
Notifications
You must be signed in to change notification settings - Fork 8
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
Adds ls and deactivate subcommands to dds motd and displaying of multiple active motds #1212
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1212 +/- ##
==========================================
+ Coverage 86.05% 86.13% +0.07%
==========================================
Files 29 29
Lines 3686 3707 +21
==========================================
+ Hits 3172 3193 +21
Misses 514 514
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@valyo Would you like to have a look again before I merge or are you satisfied? |
Looks good, if you don't want me to explicitly test it, then go on and merge 👍 |
Great! Nah you've added the tests and I've also tried it out. |
Adds ls and deactivate subcommands to dds motd
and displaying of multiple active motds
works with ScilifelabDataCentre/dds_cli#505
Please include the following in this section
Fixes # (issue)
Type of change
Checklist:
Please delete options that are not relevant.
Formatting and documentation
Tests