Skip to content

Add info about new SDK to the top of README #673

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

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

patrickloeber
Copy link
Collaborator

@patrickloeber patrickloeber commented Feb 6, 2025

Another update to improve #672 by adding a clear note in the top that the new SDK should be used

Copy link
Collaborator

@MarkDaoust MarkDaoust left a comment

Choose a reason for hiding this comment

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

  1. This is based on the wrong version, makes the diff hard to read, and that will be a merge conflict before submit. Please fix.

  2. Any change her should be mirrored to the copy in the website: https://ai.google.dev/gemini-api/docs/migrate (hit "login" [top right] then "edit" [pencil, center right] to edit.)

  3. We're intentionally not calling this deprecated until we announce a shutdown date for the gemini-1.5 series models.

@patrickloeber patrickloeber force-pushed the patloeber/readme-with-migrationguide branch from 81eda54 to 75a2b33 Compare February 6, 2025 20:25
@patrickloeber patrickloeber marked this pull request as ready for review February 6, 2025 20:26
@github-actions github-actions bot added status:awaiting review PR awaiting review from a maintainer component:python sdk Issue/PR related to Python SDK labels Feb 6, 2025
@patrickloeber
Copy link
Collaborator Author

good points! I kept it simple and now only added an info box in the top

@patrickloeber patrickloeber changed the title Update README Add info about new SDK to the top of README Feb 6, 2025
@patrickloeber patrickloeber force-pushed the patloeber/readme-with-migrationguide branch from 75a2b33 to 07d22c3 Compare February 18, 2025 15:02
@MarkDaoust MarkDaoust merged commit a7e1928 into main Feb 18, 2025
5 checks passed
@github-actions github-actions bot removed the status:awaiting review PR awaiting review from a maintainer label Feb 18, 2025
@MarkDaoust MarkDaoust mentioned this pull request Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:python sdk Issue/PR related to Python SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants