Giter Site home page Giter Site logo

Fixing Google AIP api linter about opi-api HOT 6 CLOSED

glimchb avatar glimchb commented on August 10, 2024
Fixing Google AIP api linter

from opi-api.

Comments (6)

glimchb avatar glimchb commented on August 10, 2024

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.

glimchb avatar glimchb commented on August 10, 2024

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.

glimchb avatar glimchb commented on August 10, 2024

0158 fixed by #203

from opi-api.

glimchb avatar glimchb commented on August 10, 2024

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.

glimchb avatar glimchb commented on August 10, 2024

Fixed by #317:
Now left:

+--------------------------+------------------+----------------+
|           RULE           | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------+------------------+----------------+
| core::0192::has-comments |              408 |              9 |
+--------------------------+------------------+----------------+

from opi-api.

glimchb avatar glimchb commented on August 10, 2024

Fixed by #346:
Now left:

+--------------------------+------------------+----------------+
|           RULE           | TOTAL VIOLATIONS | VIOLATED FILES |
+--------------------------+------------------+----------------+
| core::0192::has-comments |              272 |              9 |
+--------------------------+------------------+----------------+

from opi-api.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.