aep-dev / api-linter

A linter for APIs defined in protocol buffers.
Apache License 2.0
0 stars 1 forks source link

Remove AEP-128 rules #55

Closed rambleraptor closed 5 months ago

rambleraptor commented 5 months ago

Fixes #9

This removes all rules for AIP-128 (Declarative-friendly). None of these are in use for the AEPs.

toumorokoshi commented 5 months ago

I haven't stated this directly, but I was kind of wondering if we wanted to keep the "declarative-friendly" annotation around?

I never liked the fact that we had good guidance move into other AEPs, but OTOH I don't think we can get all the good declarative-friendly guidance as MUST. We may need to conditionally enable some rules, and we'll need some flag to detect that we want those rules on.

WDYT?

rambleraptor commented 5 months ago

Couple thoughts: