Skip to content
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.

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

Visibility proto lint fix #7

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

erademacher
Copy link

No description provided.

isbang and others added 8 commits August 31, 2022 00:03
…tem#2867)

* fix: support for oneof fields in request bodies

* chore: run bazelisk

* chore: update buf.yaml

* fix: bazelisk
…when user provided wrong method (grpc-ecosystem#2870)

* add failing test for DELETE /foo/bar:archive when method is wrong

The method the user meant was POST, which is supported. DELETE is not
supported, but I believe this should be NotImplemented, not NotFound.

* calculate verb in each handler
Uses the slimmer version of googleapis. See
https://docs.buf.build/faq#googleapis-failure
for more information.
…messages

Addresses issue grpc-ecosystem#2691. Skip hidden methods in
findServicesMessagesAndEnumerations.
@erademacher erademacher closed this Sep 8, 2022
@erademacher erademacher deleted the hide_unused_messages branch September 8, 2022 22:50
@erademacher erademacher restored the hide_unused_messages branch September 8, 2022 22:51
@erademacher erademacher reopened this Sep 8, 2022
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