[WIP] Conduct thorough review of codebase for improvements

[Copilot]

• Create proper Python package structure with init.py files
• Implement core Vehicle connection and management module
• Add telemetry data handling and attribute access
• Implement command execution (arm, takeoff, land, goto)
• Add mission management functionality
• Create comprehensive error handling and logging
• Add configuration management module
• Write unit tests for all modules
• Create integration tests
• Add example scripts for common use cases
• Write comprehensive README with installation and usage
• Add requirements.txt with dependencies
• Create setup.py for package installation
• Add .gitignore for Python projects
• Create contributing guidelines
• Add CI/CD configuration
• Write API documentation
• Fix flake8 configuration issues (removed invalid comments in ignore list)
• Update Python version support (removed 3.7, added 3.8-3.12)
• Auto-format code with black
• Remove unused imports

Original prompt

Conduct a deep and thorough review of this repo's codebase and provide suggestions to make the code functional, and more robust

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[cywf]

LGTM!

[cywf]

@copilot could you please fix the failing CI checks? The flake8 linting job is erroring because the ignore list contains an invalid character '#' that doesn't match the expected [A-Z][0-9]* format. Removing or correcting that entry should allow flake8 to pass. Additionally, the Ubuntu jobs targeting Python 3.7 are failing because Python 3.7 isn't available on Ubuntu 24.04. Consider removing Python 3.7 from the test matrix or using a supported OS for that version. Once these issues are addressed, please re-run the checks so they pass. Thanks!

[Copilot]

@copilot could you please fix the failing CI checks? The flake8 linting job is erroring because the ignore list contains an invalid character '#' that doesn't match the expected [A-Z][0-9]* format. Re...

Fixed both issues in commit 7d98a2f:

1. Flake8 configuration: Removed inline comments from the ignore list in setup.cfg (lines 12-13). The # character was causing the invalid format error.

2. Python version support:

   • Removed Python 3.7 from the test matrix in .github/workflows/tests.yml
   • Added Python 3.12 support
   • Updated setup.py to reflect Python 3.8+ requirement
   • Updated mypy configuration to use Python 3.8

Additionally, I ran black to auto-format all code and removed unused imports, which should help all linting checks pass cleanly.

[cywf]

LGTM!