Skip to content

CLI: move back to argparse #28

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

Merged
merged 1 commit into from
Oct 31, 2024
Merged

CLI: move back to argparse #28

merged 1 commit into from
Oct 31, 2024

Conversation

jaagut
Copy link
Member

@jaagut jaagut commented Oct 31, 2024

This pull request includes several changes to improve the ddlitlab2024 project. The most significant updates involve replacing the Tap library with argparse for command-line argument parsing, updating the database path handling to use pathlib.Path, and refactoring the main function to support new subcommands.

Command-line argument parsing improvements:

  • Replaced Tap with argparse for more flexible and standard command-line argument parsing in ddlitlab2024/dataset/cli.py. [1] [2]

Database path handling:

  • Changed the database path handling from os.path to pathlib.Path in ddlitlab2024/__init__.py and ddlitlab2024/dataset/db.py. [1] [2]

Main function refactoring:

  • Updated the main function in ddlitlab2024/dataset/main.py to handle new subcommands and improve session management.

Additional imports:

  • Added necessary imports in various files to support the new changes. [1] [2] [3]

@jaagut jaagut requested review from texhnolyze and Flova October 31, 2024 16:37
@jaagut jaagut self-assigned this Oct 31, 2024
@texhnolyze texhnolyze merged commit 5bcdd48 into main Oct 31, 2024
0 of 2 checks passed
@texhnolyze texhnolyze deleted the feature/cli_use_argparse branch October 31, 2024 16:40
Flova pushed a commit that referenced this pull request Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants