Skip to content

feat: add switch to use put as default update verb#770

Merged
baywet merged 8 commits intomainfrom
feat/use-put-for-update
Mar 19, 2026
Merged

feat: add switch to use put as default update verb#770
baywet merged 8 commits intomainfrom
feat/use-put-for-update

Conversation

@gavinbarron
Copy link
Contributor

No description provided.

@gavinbarron gavinbarron requested a review from a team as a code owner January 9, 2026 19:45
MIchaelMainer
MIchaelMainer previously approved these changes Jan 13, 2026
baywet
baywet previously approved these changes Jan 13, 2026
@baywet baywet requested a review from MIchaelMainer January 13, 2026 12:51
@baywet
Copy link
Member

baywet commented Jan 13, 2026

@copilot add unit tests for this new behaviour

Copy link
Contributor

Copilot AI commented Jan 13, 2026

@baywet I've opened a new pull request, #771, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits January 13, 2026 12:59
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
Add unit tests for UseHttpPutForUpdate setting
baywet
baywet previously approved these changes Jan 13, 2026
@sonarqubecloud

This comment was marked as outdated.

@sonarqubecloud
Copy link

Copy link

@adrian05-ms adrian05-ms left a comment

Choose a reason for hiding this comment

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

SonarCloud build seems to be failing due to a config issue, should we address this in a different PR?

@baywet baywet merged commit ed79e42 into main Mar 19, 2026
41 of 42 checks passed
@baywet baywet deleted the feat/use-put-for-update branch March 19, 2026 15:18
baywet added a commit that referenced this pull request Mar 19, 2026
feat: add switch to use put as default update verb
@baywet
Copy link
Member

baywet commented Mar 19, 2026

SonarCloud build seems to be failing due to a config issue, should we address this in a different PR?

@adrian05-ms the analysis was configured in automatic mode for some reason which was conflicting with the CI. Not sure what happened but I flipped the setting back. Hopefully that solves it.

baywet added a commit that referenced this pull request Mar 19, 2026
Merge pull request #770 from microsoft/feat/use-put-for-update
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.

5 participants