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

Update UG for addL and addG commands #61

Merged
merged 4 commits into from
Oct 17, 2023

Conversation

MadLamprey
Copy link
Collaborator

No description provided.

Misra Aditya added 4 commits October 16, 2023 15:42
* 'master' of https://github.com/AY2324S1-CS2103T-W09-4/tp: (33 commits)
  Fix nits
  CI: Improve test coverage
  Fix search by status
  Update  checkstyle issues
  CI: improve code coverage
  Update the sort method for List Command
  Fix checkstyle
  Update tests for search
  Update List Command to implement sort
  bug fixing
  Update Person
  Feat: Implement search by name and status
  Status class: add isValidStatus method and message_constraint
  Update LinkedInTest and GithubTest
  Add AddLCommandTest, AddGCommandTest, LinkedInTest and GithubTest
  Update AddL and AddG command
  Add basic structure for AddL and AddG commands.
  updated setCommand Test
  checkstyleTests
  added tests
  ...
* 'master' of https://github.com/AY2324S1-CS2103T-W09-4/tp:
  Create Test for View Command
  Checkstyle fix
  Add view command
* master: (36 commits)
  Create Test for View Command
  Checkstyle fix
  Add view command
  Fix nits
  CI: Improve test coverage
  Fix search by status
  Update  checkstyle issues
  CI: improve code coverage
  Update the sort method for List Command
  Fix checkstyle
  Update tests for search
  Update List Command to implement sort
  bug fixing
  Update Person
  Feat: Implement search by name and status
  Status class: add isValidStatus method and message_constraint
  Update LinkedInTest and GithubTest
  Add AddLCommandTest, AddGCommandTest, LinkedInTest and GithubTest
  Update AddL and AddG command
  Add basic structure for AddL and AddG commands.
  ...
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 a1fd88b into AY2324S1-CS2103T-W09-4:master Oct 17, 2023
3 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.

2 participants