-
Notifications
You must be signed in to change notification settings - Fork 59
feat(ng-dev): integrate AI tooling into ng-dev #2863
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
Conversation
josephperrott
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM
We may need to allow users to provide the auth key via an input flag as some team members prefer not to store keys like this as a env variable, but we can also wait until we actually have that feedback.
Just a few little comments to look at and we can try to get it merged in.
Integrates the AI tooling prototype into `ng-dev`.
|
@josephperrott I've addressed the feedback. I've also added the option to pass the API key as a flag, rather than using the environment variable. |
|
This PR was merged into the repository by commit b3127a1. The changes were merged into the following branches: main |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Integrates the AI tooling prototype into
ng-dev.