gradle-baseline icon indicating copy to clipboard operation
gradle-baseline copied to clipboard

Add `org.mockito.ArgumentMatchers.*` to `AvoidStaticImport` exclusions

Open MGHawes opened this issue 1 year ago • 2 comments

Before this PR

  • We did not permit any() or eq(...) static imports

After this PR

  • Add org.mockito.ArgumentMatchers.* to AvoidStaticImport exclusions
  • Is in-line with the other suppressions here for Mockito and AssertJ

==COMMIT_MSG== Add org.mockito.ArgumentMatchers.* to AvoidStaticImport exclusions ==COMMIT_MSG==

MGHawes avatar Sep 30 '24 18:09 MGHawes

Generate changelog in changelog/@unreleased

What do the change types mean?
  • feature: A new feature of the service.
  • improvement: An incremental improvement in the functionality or operation of the service.
  • fix: Remedies the incorrect behaviour of a component of the service in a backwards-compatible way.
  • break: Has the potential to break consumers of this service's API, inclusive of both Palantir services and external consumers of the service's API (e.g. customer-written software or integrations).
  • deprecation: Advertises the intention to remove service functionality without any change to the operation of the service itself.
  • manualTask: Requires the possibility of manual intervention (running a script, eyeballing configuration, performing database surgery, ...) at the time of upgrade for it to succeed.
  • migration: A fully automatic upgrade migration task with no engineer input required.

Note: only one type should be chosen.

How are new versions calculated?
  • ❗The break and manual task changelog types will result in a major release!
  • 🐛 The fix changelog type will result in a minor release in most cases, and a patch release version for patch branches. This behaviour is configurable in autorelease.
  • ✨ All others will result in a minor version release.

Type

  • [ ] Feature
  • [ ] Improvement
  • [ ] Fix
  • [ ] Break
  • [ ] Deprecation
  • [ ] Manual task
  • [ ] Migration

Description Add org.mockito.ArgumentMatchers.* to AvoidStaticImport exclusions

Check the box to generate changelog(s)

  • [ ] Generate changelog entry

changelog-app[bot] avatar Sep 30 '24 18:09 changelog-app[bot]

We already suppress AvoidStaticImport in test files. So this PR shouldn't be necessary.

https://github.com/palantir/gradle-baseline/blob/28f8b665dd7234dcd161efb8114632c8b8b2fe3b/gradle-baseline-java-config/resources/checkstyle/checkstyle-suppressions.xml#L13

pkoenig10 avatar Sep 30 '24 18:09 pkoenig10

This PR has been automatically marked as stale because it has not been touched in the last 14 days. If you'd like to keep it open, please leave a comment or add the 'long-lived' label, otherwise it'll be closed in 7 days.

stale[bot] avatar Jun 27 '25 04:06 stale[bot]