Skip to content

Comments

feat: add Configuration settings#82

Merged
averikitsch merged 3 commits intomainfrom
settings-env
Jan 28, 2026
Merged

feat: add Configuration settings#82
averikitsch merged 3 commits intomainfrom
settings-env

Conversation

@averikitsch
Copy link
Contributor

No description provided.

Copy link
Contributor

@Yuan325 Yuan325 left a comment

Choose a reason for hiding this comment

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

Assuming that these are all that we need.

Seems like there are other optional parameters in the bigquery prebuilt source: https://github.com/googleapis/genai-toolbox/blob/main/internal/prebuiltconfigs/tools/bigquery.yaml

Copy link
Contributor

@Yuan325 Yuan325 left a comment

Choose a reason for hiding this comment

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

Assuming that these are all that we need.

Seems like there are other optional parameters in the bigquery prebuilt source: https://github.com/googleapis/genai-toolbox/blob/main/internal/prebuiltconfigs/tools/bigquery.yaml

Copy link
Contributor

@Yuan325 Yuan325 left a comment

Choose a reason for hiding this comment

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

Assuming that these are all that we need.

Seems like there are other optional parameters in the bigquery prebuilt source: https://github.com/googleapis/genai-toolbox/blob/main/internal/prebuiltconfigs/tools/bigquery.yaml

@averikitsch
Copy link
Contributor Author

Assuming that these are all that we need.

Seems like there are other optional parameters in the bigquery prebuilt source: https://github.com/googleapis/genai-toolbox/blob/main/internal/prebuiltconfigs/tools/bigquery.yaml

I've added an issue to support the additional fields.

@averikitsch averikitsch merged commit ba8aba6 into main Jan 28, 2026
7 checks passed
@averikitsch averikitsch deleted the settings-env branch February 18, 2026 17:58
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.

4 participants