Skip to content

apply builder pattern to OllamaApi #2634

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

dev-jonghoonpark
Copy link
Contributor

@dev-jonghoonpark dev-jonghoonpark commented Apr 3, 2025

apply Builder pattern to OllamaApi to provide a consistent development design

related: #2164, #2817

@ThomasVitale
Copy link
Contributor

@dev-jonghoonpark thanks for your contribution!

@dev-jonghoonpark dev-jonghoonpark force-pushed the apply-builder-pattern-to-OllamaApi branch from 1ae9c99 to 9b9424b Compare April 26, 2025 14:05
@ilayaperumalg ilayaperumalg added this to the 1.0.0-RC1 milestone Apr 28, 2025
Signed-off-by: jonghoon park <dev@jonghoonpark.com>
@dev-jonghoonpark dev-jonghoonpark force-pushed the apply-builder-pattern-to-OllamaApi branch from 9b9424b to 3a483f7 Compare April 28, 2025 19:16
@dev-jonghoonpark
Copy link
Contributor Author

conflict resolved.

@ilayaperumalg ilayaperumalg self-assigned this Apr 29, 2025
@ilayaperumalg
Copy link
Member

@dev-jonghoonpark Thanks for adding the builder pattern to OllamaApi. Rebased and merged as d48f4f6

@ilayaperumalg ilayaperumalg modified the milestones: 1.0.0-RC1, 1.0.0-M8 Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants