Skip to content

Delete the generated spec assets in the agent SDK#773

Merged
gspencergoog merged 2 commits intogoogle:mainfrom
nan-yu:remove-schema-copy
Mar 4, 2026
Merged

Delete the generated spec assets in the agent SDK#773
gspencergoog merged 2 commits intogoogle:mainfrom
nan-yu:remove-schema-copy

Conversation

@nan-yu
Copy link
Collaborator

@nan-yu nan-yu commented Mar 4, 2026

Description

#760 accidentally checked in obsolete and generated assets.
This commit removes them and also updates the git ignore file to ignore them when checking in code.

Pre-launch Checklist

If you need help, consider asking for advice on the discussion board.

google#760 accidentally checked in obsolete
and generated assets.
This commit removes them and also updates the git ignore file to ignore
them when checking in code.
@nan-yu nan-yu force-pushed the remove-schema-copy branch from 4537e5c to a47426c Compare March 4, 2026 21:21
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request addresses an issue where generated and obsolete assets were accidentally checked into the repository. The changes involve removing these JSON asset files from the a2a_agents directory and updating the root .gitignore file to prevent them from being checked in again. The local .gitignore from the Python agent SDK is also removed as its rule has been centralized. My review suggests a small improvement to the new .gitignore rules to make them more generic and future-proof.

yourkarma6788

This comment was marked as spam.

@google google deleted a comment from yourkarma6788 Mar 4, 2026
@gspencergoog gspencergoog merged commit bca0403 into google:main Mar 4, 2026
7 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in A2UI Mar 4, 2026
@nan-yu nan-yu deleted the remove-schema-copy branch March 5, 2026 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants