[PR #86] Refactor: Use argparse for command-line argument parsing #131

New issue

Open

opened 2026-05-07 00:18:49 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 00:18:49 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/IAmTomShaw/f1-race-replay/pull/86
Author: @Mirza-Samad-Ahmed-Baig
Created: 12/19/2025
Status: 🔄 Open

Base: main ← Head: feature/argparse-refactor

📝 Commits (1)

f92f028 Refactor: Use argparse for command-line argument parsing

📊 Changes

1 file changed (+28 additions, -32 deletions)

View changed files

📝 main.py (+28 -32)

📄 Description

This PR refactors main.py to use argparse for command-line argument parsing, replacing the manual parsing of sys.argv.

Advantages:

Improved Readability & Maintainability: argparse provides a cleaner, more organized, and easier-to-understand way to define arguments, types, and descriptions.
Automatic Help Messages: argparse automatically generates help messages (e.g., h or --help) for a better user experience.
Robust Error Handling: argparse provides informative error messages for invalid arguments, reducing the need for custom error handling.
Easier to Extend: Adding new arguments is now simpler and less error-prone.

Reasoning:

The previous sys.argv implementation was less robust and harder to maintain. This refactoring improves code quality and user-friendliness.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/IAmTomShaw/f1-race-replay/pull/86 **Author:** [@Mirza-Samad-Ahmed-Baig](https://github.com/Mirza-Samad-Ahmed-Baig) **Created:** 12/19/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feature/argparse-refactor` --- ### 📝 Commits (1) - [`f92f028`](https://github.com/IAmTomShaw/f1-race-replay/commit/f92f028da5f1fa3bb0ddb9e97c9e94e807eec8c9) Refactor: Use argparse for command-line argument parsing ### 📊 Changes **1 file changed** (+28 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `main.py` (+28 -32) </details> ### 📄 Description This PR refactors main.py to use argparse for command-line argument parsing, replacing the manual parsing of sys.argv. Advantages: * Improved Readability & Maintainability: argparse provides a cleaner, more organized, and easier-to-understand way to define arguments, types, and descriptions. * Automatic Help Messages: argparse automatically generates help messages (e.g., h or --help) for a better user experience. * Robust Error Handling: argparse provides informative error messages for invalid arguments, reducing the need for custom error handling. * Easier to Extend: Adding new arguments is now simpler and less error-prone. Reasoning: The previous sys.argv implementation was less robust and harder to maintain. This refactoring improves code quality and user-friendliness. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>