Skip to content

Conversation

@Hyune-c
Copy link
Contributor

@Hyune-c Hyune-c commented Oct 5, 2024

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits ✅
  • Add/Update unit tests as needed ✅
  • Run a build and make sure all tests pass prior to submission ✅

I updated the AnthropicChatModel by changing the anthropicApi field from public to private.

  • Ensures consistency since all other major variables are private
  • Aligns with how other AI provider models handle their APIs
  • Verified that anthropicApi is only used internally, avoiding external issues

Thank you for reviewing this contribution.

@markpollack markpollack self-assigned this Oct 6, 2024
@markpollack
Copy link
Member

thanks, merged in e123721

@markpollack markpollack closed this Oct 6, 2024
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