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

Branch tag list #193

Merged

Conversation

daryltay415
Copy link

No description provided.

# Conflicts:
#	omni.txt
#	src/main/java/seedu/omnitravel/errorhandlers/CheckParameters.java
#	src/main/java/seedu/omnitravel/parser/Parser.java
#	src/test/java/seedu/omnitravel/OmniTravelTest.java
* Checks if a string contains all the words
* @param input The input String
*/
public static void containsWords(String input) throws OmniException{

Choose a reason for hiding this comment

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

Good job in identifying the keywords for duration

* @param list The travel activity list
* @throws OmniException Throws an exception when the command is in an invalid format
*/
public static void listTagsCommand(String[] command, TravelActivityList list) throws OmniException{

Choose a reason for hiding this comment

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

LGTM!

@EugeneChanJiajun EugeneChanJiajun merged commit 162066f into AY2324S2-CS2113-T12-4:master Apr 6, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment