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

buf lint uses out of date field behaviour proto. #2474

Closed
DavyJ0nes opened this issue Oct 3, 2023 · 1 comment
Closed

buf lint uses out of date field behaviour proto. #2474

DavyJ0nes opened this issue Oct 3, 2023 · 1 comment

Comments

@DavyJ0nes
Copy link

I recently came across inconsistencies between buf lint and google's api-linter due to the field_behaviour.proto being out of date in buf.

There is a requirement in the api-linter to have google.api.field_behavior = IDENTIFIER on the name field of a message but when setting this and running buf lint I get the following error:

Foo.name: option (google.api.field_behavior): enum google.api.FieldBehavior has no value named IDENTIFIER

field_behaviour.proto was updated to add this new enum value on 22/08/23 so it looks like buf needs to update its dependencies?

I hope this is enough information to go on and reproduce the issue, if not then let me know.

DavyJ0nes pushed a commit to einride/sage that referenced this issue Oct 3, 2023
Updates api-linter to version v1.56.1 as v.1.57.0 introduced a new
requirement for (google.api.field_behavior) = IDENTIFIER to be set on
the name field of a proto resource.

There is a conflict between buf lint and api-linter due to an update to
field_behaviour.proto. An issue has been opened buf as fix is on their side:
bufbuild/buf#2474
DavyJ0nes pushed a commit to einride/sage that referenced this issue Oct 3, 2023
Updates api-linter to version v1.56.1 as v.1.57.0 introduced a new
requirement for (google.api.field_behavior) = IDENTIFIER to be set on
the name field of a proto resource.

There is a conflict between buf lint and api-linter due to an update to
field_behaviour.proto. An issue has been opened buf as fix is on their side:
bufbuild/buf#2474
@DavyJ0nes
Copy link
Author

This is an issue on my side with buf dependencies. Updating with buf mod update will bring the latest googleapis version to include the IDENTIFER enum. Closing...

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

No branches or pull requests

1 participant