Skip to content
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

Added more tests #179

Merged
merged 6 commits into from
Nov 12, 2023
Merged

Conversation

MadLamprey
Copy link
Collaborator

No description provided.

Misra Aditya added 6 commits November 10, 2023 11:31
* master:
  Remove TODO
  Fix tag, status, and name parser bug
  Add delete command parser tests
  Added planned enhancements for export into the User Guide
  Add testing for DeleteCommand
  Updated the User Guide to enhance clarity for users
* 'master' of https://github.com/AY2324S1-CS2103T-W09-4/tp:
  Match the features explanation
  Edit user guide
  Update DG
  Update user guide for delete command
  Add prefix table, remove known issues
  Update DG
  Update DG
  Remove duplicate events testing [not allowed in feature freeze]
  Fix bugs
  Add tests
* master: (32 commits)
  Match the features explanation
  Edit user guide
  Fix nits
  Improve value proposition in introduction
  Fix checkstyle: remove unused import
  Replace removeExtraSpace method with a simple trim()
  Fix color coding for 6th tag category
  Bugfix: use of multiple tags with categories in create, edit and add
  Update DG
  Update user guide for delete command
  Add prefix table, remove known issues
  Fix color coding of tags
  Update DG
  Update DG
  Update PPP
  Update PPP
  Fix CI build
  Update PPP-draft
  Updated PPP to reflect UG
  Remove duplicate events testing [not allowed in feature freeze]
  ...
Copy link

codecov bot commented Nov 12, 2023

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Files Coverage Δ
src/main/java/seedu/address/model/EventBook.java 87.09% <ø> (+2.72%) ⬆️
...ava/seedu/address/model/event/UniqueEventList.java 84.37% <ø> (+2.55%) ⬆️
...va/seedu/address/logic/commands/GithubCommand.java 22.72% <33.33%> (+22.72%) ⬆️
.../seedu/address/logic/commands/LinkedInCommand.java 22.72% <33.33%> (+22.72%) ⬆️

... and 3 files with indirect coverage changes

📢 Thoughts on this report? Let us know!

Copy link

@sk2001git sk2001git left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MadLamprey MadLamprey merged commit c26c1a9 into AY2324S1-CS2103T-W09-4:master Nov 12, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants