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

Updated Saq AnswerChecker, added Saq AnswerCheckerTest and added Saq questions to SampleDataUtil #155

Merged

Conversation

neoshh
Copy link

@neoshh neoshh commented Nov 7, 2019

No description provided.

* master:
  update autocomplete to include history
  Edit for checkstyle
  Refactor 4
  added test for McqInputCommand
  Refactor part 3
  Refactoring part 2
  Refactor AdressBook to RevisionTool
  Added test for Restore and StartQuiz
  Add in difficulty scores for statistics
  Resolve delete duplicate questions issue
  updateHistory, handleStats, handleHistory
  Add sample history
  Adds new line at EOF
  Troubleshooting stats and history commands
  Update history and stats functionalities
  Link up history and stats with storage, model and logic
  Add stats and history command
  Checkstyle
  Add stats command
@neoshh neoshh changed the title Added Saq AnswerCheckerTest and added Saq questions to SampleDataUtil Updated Saq AnswerChecker, added Saq AnswerCheckerTest and added Saq questions to SampleDataUtil Nov 7, 2019
Copy link

@jxofficial jxofficial left a comment

Choose a reason for hiding this comment

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

LGTM, I'll go ahead and merge

@jxofficial jxofficial merged commit 82119f7 into AY1920S1-CS2103-F10-3:master Nov 9, 2019
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

3 participants