-
Notifications
You must be signed in to change notification settings - Fork 116
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
PR #116
base: master
Are you sure you want to change the base?
PR #116
Conversation
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.
Overall, I found your code simple to read. However I do note the lack of comments and hope to see more!
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.
Looks good!
For improving code quality you can add more functionalities like managing deadline for a task that must be completed by a specific date or an Event task which occurs during a specific time range. |
Add more comments for more readability |
Proceeded as per description- Move duke out of package
Main file added in another increment (Level-5)
updated with airborder_exception
with command-line arguments to accept file path from user
deleted to avoid merge conflicts
…o branch-Level-6
Redundant file-naming error
No description provided.