Comments (6)
with latest merged code https://github.com/opiproject/opi-api/blob/main/storage/Makefile#L18
docker run --user=$$(id -u):$$(id -g) --rm --entrypoint=sh -v "${PWD}/../common/v1":/common -v "${PWD}"/v1alpha1/:/out -w /out ghcr.io/docker-multiarch/google-api-linter:1.36.0 -c "api-linter -I /common /out/*.proto --output-format github --disable-rule=core::0192::has-comments --disable-rule=core::0136 --disable-rule=core::0127 --set-exit-status"
still ignoring 3 rules that need to re-enable them:
core::0192::has-comments
core::0136
core::0127
which are:
| core::0136::http-method | 12 | 3 |
| core::0136::http-body | 12 | 3 |
| core::0127::resource-name-extraction | 12 | 3 |
| core::0127::http-annotation | 24 | 6 |
| core::0136::http-uri-suffix | 30 | 3 |
| core::0192::has-comments | 282 | 6 |
from opi-api.
after #202
+--------------------------------------------+------------------+----------------+
| RULE | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------------------------+------------------+----------------+
| core::0133::http-body | 3 | 2 |
| core::0132::request-unknown-fields | 4 | 2 |
| core::0135::request-name-required | 6 | 3 |
| core::0133::request-resource-behavior | 6 | 3 |
| core::0134::request-unknown-fields | 6 | 3 |
| core::0134::request-mask-required | 6 | 3 |
| core::0133::request-parent-required | 6 | 3 |
| core::0134::method-signature | 6 | 3 |
| core::0134::http-body | 6 | 3 |
| core::0158::request-page-size-field | 6 | 3 |
| core::0131::method-signature | 6 | 3 |
| core::0131::request-name-required | 6 | 3 |
| core::0134::request-resource-field | 6 | 3 |
| core::0133::request-resource-field | 6 | 3 |
| core::0131::http-uri-name | 6 | 3 |
| core::0133::http-uri-parent | 6 | 3 |
| core::0132::request-parent-required | 6 | 3 |
| core::0158::request-page-token-field | 6 | 3 |
| core::0158::response-next-page-token-field | 6 | 3 |
| core::0135::method-signature | 6 | 3 |
| core::0158::response-plural-first-field | 6 | 3 |
| core::0134::http-uri-name | 6 | 3 |
| core::0132::response-unknown-fields | 6 | 3 |
| core::0133::method-signature | 6 | 3 |
| core::0135::http-uri-name | 6 | 3 |
| core::0131::request-unknown-fields | 7 | 3 |
| core::0135::request-unknown-fields | 7 | 3 |
| core::0127::resource-name-extraction | 12 | 3 |
| core::0127::http-annotation | 24 | 6 |
| core::0192::has-comments | 281 | 6 |
+--------------------------------------------+------------------+----------------+
from opi-api.
0158
fixed by #203
from opi-api.
Fixed by #218 :
- [storage] [lint] Re-enable rule=core::0135
- [storage] [lint] Re-enable rule=core::0131
- [storage] [lint] Re-enable rule=core::0132
- [storage] [lint] Re-enable rule=core::0134
- [storage] [lint] Re-enable rule=core::0133
Now left:
+--------------------------------------+------------------+----------------+
| RULE | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------------------+------------------+----------------+
| core::0156::forbidden-methods | 11 | 7 |
| core::0127::http-annotation | 12 | 7 |
| core::0127::resource-name-extraction | 22 | 7 |
| core::0192::has-comments | 380 | 8 |
+--------------------------------------+------------------+----------------+
from opi-api.
Fixed by #317:
Now left:
+--------------------------+------------------+----------------+
| RULE | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------+------------------+----------------+
| core::0192::has-comments | 408 | 9 |
+--------------------------+------------------+----------------+
from opi-api.
Fixed by #346:
Now left:
+--------------------------+------------------+----------------+
| RULE | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------+------------------+----------------+
| core::0192::has-comments | 272 | 9 |
+--------------------------+------------------+----------------+
from opi-api.
Related Issues (20)
- Malformed `go_package` option in `*.proto`, please specify
- consider adding nvme_controller_suspend/nvme_controller_resume
- [storage] NvmeRemoteController* -> NvmeBackendController* HOT 1
- nvme: consider combining `FabricsEndpoint` and `FabricsPath`
- nvme namespace: `volume_name_ref` missing type
- Remove resource reference annotations from names of resources
- Keep bufbuild-go-gen image under opiproject organization repo HOT 2
- Enable BUF linting and generation on all API folders
- Add and Cleanup COMMENT support in BUF linting checking
- Add versions to the generator functions in buf.gen.yaml
- Add version tracking for go-grpc to renovate for the buf.gen.yaml
- [network]: Define Port configuration API
- [network] Define virtual port/device API HOT 1
- Include https://github.com/googleapis/api-linter in protobuf checking.
- [security] Update IPSec protobuf API to CRUD model HOT 1
- [network] Define L2 switching support APIs
- Use signed integers in evpn gw API protobufs
- configuring 1M IPsec tunnels takes 45 minutes , configuring 32M imposible
- Find closer integration between google api linter and buf
- Align Makefiles across folders
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from opi-api.