Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: added the ability to set a user default AI provider #427

Merged
merged 3 commits into from
May 16, 2023
Merged

feat: added the ability to set a user default AI provider #427

merged 3 commits into from
May 16, 2023

Conversation

AlexsJones
Copy link
Member

Closes #

📑 Description

✅ Checks

  • [x ] My pull request adheres to the code style of this project
  • [ x] My code requires changes to the documentation
  • [ x] I have updated the documentation as required
  • [ x] All the tests have passed

ℹ Additional Information

#423 #424

@AlexsJones AlexsJones requested review from a team as code owners May 16, 2023 08:08
@AlexsJones AlexsJones enabled auto-merge (squash) May 16, 2023 08:18
@AlexsJones AlexsJones disabled auto-merge May 16, 2023 08:18
matthisholleville
matthisholleville reviewed May 16, 2023
View reviewed changes
Copy link
Contributor

@matthisholleville matthisholleville left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work. Left two comments

pkg/analysis/analysis.go Show resolved Hide resolved
pkg/analysis/output.go Show resolved Hide resolved
@AlexsJones AlexsJones mentioned this pull request May 16, 2023
Closed
4 tasks
@AlexsJones
feat: added the ability to set a user default AI provider
e042c64 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
feat: added the ability to set a user default AI provider
d5dfcf0 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones
chore: added provider to json output
1658fc2 
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones AlexsJones enabled auto-merge (squash) May 16, 2023 11:29
@AlexsJones
Copy link
Member Author

Can you approve it @matthisholleville ?

@AlexsJones AlexsJones merged commit cbe2fb4 into main May 16, 2023
7 checks passed
@AlexsJones AlexsJones deleted the feat/default-user-backend branch May 16, 2023 11:31
@github-actions github-actions bot mentioned this pull request May 16, 2023
Merged
@VaibhavMalik4187 VaibhavMalik4187 mentioned this pull request Feb 17, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthisholleville matthisholleville matthisholleville approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Backlog
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@AlexsJones @matthisholleville