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.

Sign up for GitHub

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

Jump to bottom

fix(AIP-135): allow required etag in Delete #1414

Merged

gcf-merge-on-green merged 1 commit into googleapis:main from noahdietz:etag-request-behavior

Aug 9, 2024

Collaborator

noahdietz commented Aug 8, 2024

Allow Standard Field etag to be REQUIRED in Standard Delete.

Updates #1395


          fix(AIP-135): allow required etag

770ca7e

noahdietz requested a review from a team as a code owner

August 8, 2024 22:29

noahdietz mentioned this pull request

AIP-154: etag field_behavior #1395

Closed

noahdietz changed the title ~~fix(AIP-135): allow required etag~~ fix(AIP-135): allow required etag in Delete

noahdietz requested a review from slevenick

August 8, 2024 22:36

slevenick approved these changes

View reviewed changes

slevenick left a comment

I strongly dislike this rule, but it does match the AIP

noahdietz added the automerge label

gcf-merge-on-green bot merged commit aa9587b into googleapis:main

6 checks passed

gcf-merge-on-green bot removed the automerge label

release-please bot mentioned this pull request

chore(main): release 1.67.2 #1412

Merged

fredrikaverpil mentioned this pull request

chore(api-linter): bump, fix false warning on required etag on req einride/sage#567

Closed

gcf-merge-on-green bot pushed a commit that referenced this pull request


          chore(main): release 1.67.2 (#1412)

4eae5b0

🤖 I have created a release *beep* *boop*
---


## [1.67.2](https://github.com/googleapis/api-linter/compare/v1.67.1...v1.67.2) (2024-08-14)


### Bug Fixes

* **AIP-123:** multiword singleton reduction ([#1417](https://github.com/googleapis/api-linter/issues/1417)) ([7868552](https://github.com/googleapis/api-linter/commit/7868552ff7b27c2fa0f2ff9be3a538763f0450c5))
* **AIP-135:** allow required etag in Delete ([#1414](https://github.com/googleapis/api-linter/issues/1414)) ([aa9587b](https://github.com/googleapis/api-linter/commit/aa9587bc7184a78109f138c809baa00018ea75e9)), refs [#1395](https://github.com/googleapis/api-linter/issues/1395)
* **AIP-235:** allow hosting allow_missing ([#1416](https://github.com/googleapis/api-linter/issues/1416)) ([6bfbcdf](https://github.com/googleapis/api-linter/commit/6bfbcdfa8858ccdba98760d76e2d2a757855cc7b)), refs [#1404](https://github.com/googleapis/api-linter/issues/1404)
* exit rule if response type cannot be resolved ([#1415](https://github.com/googleapis/api-linter/issues/1415)) ([6874dab](https://github.com/googleapis/api-linter/commit/6874dabb4f0d3503f267bb0ab970d62785d12727)), refs [#1399](https://github.com/googleapis/api-linter/issues/1399)


### Documentation

* **AIP-143:** fix rule name used for implementation link ([#1411](https://github.com/googleapis/api-linter/issues/1411)) ([f9cf2eb](https://github.com/googleapis/api-linter/commit/f9cf2ebc9589abfce88317b1e3318a9e1547b41a))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet