Skip to content

Make fetch tool not read-only#1786

Merged
dgageot merged 1 commit intodocker:mainfrom
dgageot:fetch-not-readonly
Feb 19, 2026
Merged

Make fetch tool not read-only#1786
dgageot merged 1 commit intodocker:mainfrom
dgageot:fetch-not-readonly

Conversation

@dgageot
Copy link
Member

@dgageot dgageot commented Feb 19, 2026

No description provided.

@dgageot dgageot requested a review from a team as a code owner February 19, 2026 08:42
trungutt
trungutt previously approved these changes Feb 19, 2026
krissetto
krissetto previously approved these changes Feb 19, 2026
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Review Summary

No issues found in the changed code. The removal of ReadOnlyHint: true is an intentional change that aligns with the PR goal to allow the API tool to handle both read and write operations (supporting both GET and POST methods). The reformatted Title line contains no bugs.

✅ Changed code looks good!

@dgageot
Copy link
Member Author

dgageot commented Feb 19, 2026

Did I change the wrong one? It's possible...

Signed-off-by: David Gageot <david.gageot@docker.com>
@dgageot dgageot dismissed stale reviews from krissetto and trungutt via c6f48d4 February 19, 2026 08:49
@dgageot dgageot merged commit 2cceb97 into docker:main Feb 19, 2026
5 checks passed
@dgageot dgageot deleted the fetch-not-readonly branch February 27, 2026 19:37
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.

3 participants