airlift / airbase

Base POM for Airlift
Apache License 2.0
48 stars 87 forks source link

Replace checkstyle illegal import rules with RestrictImports #357

Closed wendigo closed 1 year ago

wendigo commented 1 year ago

Alternative approach to https://github.com/airlift/airbase/pull/355 as suggested by @electrum

For reference: https://github.com/skuzzle/restrict-imports-enforcer-rule

wendigo commented 1 year ago

cc @electrum @hashhar @kokosing @findepi

hashhar commented 1 year ago

errorprone profile isn't enabled by default locally so devs won't catch until these reach CI - just something to consider

wendigo commented 1 year ago

@hashhar enforcer is enabled on a default profile. I've checked that PR with the minimal set of flags

hashhar commented 1 year ago

@hashhar enforcer is enabled on a default profile. I've checked that PR with the minimal set of flags

Ah, I was confused between errorprone and enforcer plugins. 😄