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

Add logging and exceptions to help and exit command #90

Closed
wants to merge 21 commits into from

Conversation

coraleaf0602
Copy link

No description provided.

* 'master' of https://github.com/AY2425S1-CS2113-T10-2/tp: (28 commits)
  Fix CI Issue for testFind and testDelete
  Fix JUnit Expected Results
  Modify JUnit Test
  Fix Checkstyle issue
  Add exception throwing for JUnit Test
  Fix Checkstyle Issue
  :w
  fixed bug in FileHandler where duplicate of prefix variables being read
  Add annotation for methods
  Fix file directory issue
  Add JUnit test for add, list, find and delete
  merge-main
  merge-commandhandler
  merge-filehandler
  reverting
  Reapply "Merge pull request AY2425S1-CS2113-T10-2#37 from coraleaf0602/master"
  Edit the build.gradle file to set the mainClass and archiveBaseName correctly
  add Junit tests for find methods
  Modify code to align with checkstyle
  Modify gradle to allow testing
  ...

# Conflicts:
#	src/main/java/bookbob/Main.java
#	src/main/java/bookbob/functions/CommandHandler.java
#	src/main/java/bookbob/functions/FileHandler.java
#	src/test/java/bookbob/BookBobTest.java
#	src/test/java/seedu/bookbob/BookBobTest.java
#	text-ui-test/EXPECTED.TXT
#	text-ui-test/input.txt
* 'master' of https://github.com/AY2425S1-CS2113-T10-2/tp:
  each segment after an underscore must start with a lowercase letter
  Update author annotations for yentheng0110 in CommandHandler.java
  Remove duplicate testDelete
  Add Junit tests
  Implement JUnit Test for [find] command
  Implement JUnit Test for [delete] command
  Implement JUnitTest for [list] command
  Implement JUnit Test for [add] command
  Add Author to "find"
  Fix Checkstyle issues, modify testDelete() and testList() methods
  Modify testHelp() to ensure the test passed
  Add bookbob_data.txt
  Fix Checkstyle issues again
  Modify gradle.yml file
  Fix Checkstyle Issues
* 'master' of https://github.com/AY2425S1-CS2113-T10-2/tp:
  1. Improved exception handling 2. Added Test for File Handler 3. Added logger for File Handler

# Conflicts:
#	data/bookbob_data.txt
# Conflicts:
#	data/bookbob_data.txt
@coraleaf0602 coraleaf0602 changed the title Add assertions, logging and exceptions to help and exit command Add logging and exceptions to help and exit command Oct 14, 2024
This was unlinked from issues Oct 14, 2024
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.

1 participant