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

feat: add picocli command interface/logic #13

Merged
merged 18 commits into from
Oct 16, 2024
Merged

Conversation

lentidas
Copy link
Owner

This pull request adds the picocli command logic to our program.

There are still 2-3 TODOs to address, but this is a working version and solves #1 and #2.

@lentidas lentidas force-pushed the feat/add_picocli_commands branch from 07e47a6 to 198905d Compare October 16, 2024 14:59
src/main/java/ch/heigvd/dai/Main.java Outdated Show resolved Hide resolved
src/main/java/ch/heigvd/dai/commands/Root.java Outdated Show resolved Hide resolved
src/main/java/ch/heigvd/dai/utilities/FileManipulator.java Outdated Show resolved Hide resolved
src/main/java/ch/heigvd/dai/utilities/FileManipulator.java Outdated Show resolved Hide resolved
PedroAS7
PedroAS7 previously approved these changes Oct 16, 2024
* feat: add Dockerfile and GitHub workflow to publish image

* ci: update .github/workflows/builder.yaml with suggestion
* ci: add workflows to release package automatically

* fix: add missing new lines
PedroAS7
PedroAS7 previously approved these changes Oct 16, 2024
Copy link
Collaborator

@PedroAS7 PedroAS7 left a comment

Choose a reason for hiding this comment

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

Nothing changed since other PRs

@lentidas lentidas merged commit 16c3fdc into main Oct 16, 2024
3 checks passed
@lentidas lentidas deleted the feat/add_picocli_commands branch October 16, 2024 20:12
This was linked to issues Oct 17, 2024
@lentidas lentidas linked an issue Oct 17, 2024 that may be closed by this pull request
6 tasks
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.

Output the execution time Validate user input in command-line options Add command-line options
2 participants