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

[hpcman] iP #623

Open
wants to merge 83 commits into
base: master
Choose a base branch
from
Open

[hpcman] iP #623

wants to merge 83 commits into from

Conversation

hpcman
Copy link

@hpcman hpcman commented Sep 1, 2024

PacManPro

“Your mind is for having ideas, not holding them.” – David Allen (source)

PacManPro frees your mind of having to remember things you need to do. It's,

  • text-based
  • easy to learn
  • FAST SUPER FAST to use

All you need to do is,

  1. download it from here.
  2. double-click it.
  3. add your tasks.
  4. let it manage your tasks for you 😉

And it is FREE!

Features:

  • Managing tasks
  • Managing deadlines (coming soon)
  • Reminders (coming soon)

If you are a Java programmer, you can use it to practice Java too. Here's the main method:

public class Main {
    public static void main(String[] args) {
        Application.launch(MainApp.class, args);
    }
}

damithc and others added 14 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.
Copy link

@WuJinhan1 WuJinhan1 left a comment

Choose a reason for hiding this comment

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

Overall, LGTM! Just suggested that the files should be put inside a package.

import java.util.ArrayList;
import java.util.Scanner;

public class PacMan {

Choose a reason for hiding this comment

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

I like how you named your chatbot PacMan! Putting a figure behind the chatbot gives it more character.


Hello! I'm PacMan. How can I help you?
Got it. I've added this task:
[T][ ] CS2103T iP

Choose a reason for hiding this comment

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

I like how you made the tasks relevant to the class we are taking.

@@ -0,0 +1,18 @@
public class Task {
private final String taskName;

Choose a reason for hiding this comment

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

LGTM!

@@ -0,0 +1,10 @@
public class Todo extends Task {
Todo(String task) {

Choose a reason for hiding this comment

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

I like how you split up each type of task into different files, making the code really clean!

@@ -0,0 +1,13 @@
public class Deadline extends Task {

Choose a reason for hiding this comment

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

Should all of this be in a package?

hpcman and others added 30 commits September 19, 2024 12:33
The past GUI is bare bone and not visually appealing.

The GUI needs to be better for visual appeal and easier to separate the command
and response.

Let's,
* use CSS FXML to make the background and dialog box
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