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
Person -p flag for deleting #187
Person -p flag for deleting #187
Conversation
Pull Request Test Coverage Report for Build 240
💛 - Coveralls |
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.
Great use of abstraction!
*/ | ||
public class DeleteCommand extends Command { | ||
public abstract class DeleteCommand extends Command { |
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.
Wow! 😯Great idea 💡making this an abstract class! I 🙋♂️ personally wouldn't have thought 💭 of it myself! This will make implementing the delete training command a lot easier!
* A flag that marks the beginning of a type that the command is applied to. | ||
* E.g. '-p' in 'delete -p 1'. | ||
*/ | ||
public class Flag { |
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.
Great abstraction of the flag class. Will be useful for my training command as well.
This will enable @jeunhoe and I to use
delete -t
anddelete -e
flags to deleteTraining
andEvent
respectively, as thedelete
command will no longer be reserved for deleting a person.