-
Notifications
You must be signed in to change notification settings - Fork 66
Display model name used #705
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
base: main
Are you sure you want to change the base?
Conversation
|
Hey @robvanderveer @northdpole 👋 |
|
Thanks @PRAteek-singHWY, this is indeed a nuisance it seems. |
Thanks for confirming, @robvanderveer — appreciate it. |
I'\d love for this to be pre-commit hooks, using it in the github ci can be a pain since in rare cases linting can produce non-functional code We can easily make this pre-commit hooks using a pre-commit framework however, that'd be great |
@northdpole Thanks for the feedback! I completely understand the concern about CI modifying code or breaking builds. To address this, I’ve updated the PR to pivot to a local-only approach:
This creates a purely opt-in developer capability with no impact on the repository's build stability. Ready for review! |
Code was fully Cursor generated and reviewd by me