Skip to content

Simplify cli #269

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 29 commits into from
Jun 12, 2025
Merged

Simplify cli #269

merged 29 commits into from
Jun 12, 2025

Conversation

jlamypoirier
Copy link
Collaborator

✨ Description

This removes bloat and ad-hoc registries for the cli, and instead use a dynamic config class to get the exact same result in a much simpler way.

🔍 Type of change

Select all that apply:

  • 🐛 Bug fix (non-breaking change that addresses a specific issue)
  • 🚀 New feature (non-breaking change that adds functionality)
  • ⚠️ Breaking change (a change that could affect existing functionality)
  • 📈 Performance improvement/optimization (improves speed, memory usage, or efficiency)
  • 🛠️ Code refactor (non-functional changes that improve code readability, structure, etc.)
  • 📦 Dependency bump (updates dependencies, including Dockerfile or package changes)
  • 📝 Documentation change (updates documentation, including new content or typo fixes)
  • 🔧 Infrastructure/Build change (affects build process, CI/CD, or dependencies)

@jlamypoirier jlamypoirier changed the base branch from minimalistic_dynamic_classes to main May 15, 2025 15:23
@jlamypoirier jlamypoirier changed the base branch from main to minimalistic_dynamic_classes May 15, 2025 15:23
@jlamypoirier jlamypoirier marked this pull request as ready for review May 15, 2025 15:29
Base automatically changed from minimalistic_dynamic_classes to main May 21, 2025 21:07
@jlamypoirier jlamypoirier requested review from RaymondLi0 and tscholak and removed request for RaymondLi0 May 27, 2025 14:39
@jlamypoirier jlamypoirier requested a review from bigximik June 4, 2025 14:00
@jlamypoirier jlamypoirier changed the base branch from main to dynamic_transformer June 4, 2025 16:31
Base automatically changed from dynamic_transformer to main June 12, 2025 22:12
@jlamypoirier jlamypoirier merged commit b592613 into main Jun 12, 2025
1 of 2 checks passed
@jlamypoirier jlamypoirier deleted the simplify_cli branch June 12, 2025 22:14
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.

2 participants