Skip to content

[izruff] iP#626

Open
izruff wants to merge 118 commits intonus-cs2103-AY2425S1:masterfrom
izruff:master
Open

[izruff] iP#626
izruff wants to merge 118 commits intonus-cs2103-AY2425S1:masterfrom
izruff:master

Conversation

@izruff
Copy link

@izruff izruff commented Sep 2, 2024

No description provided.

damithc and others added 30 commits July 11, 2024 16:52
In build.gradle, the dependencies on distZip and/or distTar causes
the shadowJar task to generate a second JAR file for which the
mainClass.set("seedu.duke.Duke") does not take effect.
Hence, this additional JAR file cannot be run.
For this product, there is no need to generate a second JAR file
to begin with.

Let's remove this dependency from the build.gradle to prevent the
shadowJar task from generating the extra JAR file.
Certain errors can occur when user gives an incorrect or badly
formatted input, causing the program to exit. Let's handle some of
these possibilities, including,
* unknown command names, flags, or arguments
* missing or duplicated flags, arguments
* type validation for arguments
* out-of-range errors when accessing task list through `mark` or
  `unmark`
izruff and others added 30 commits September 20, 2024 16:33
Improve CommandParser code quality
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.

5 participants