Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Introduce Immutables Meta-Annotations #5494

Merged
merged 9 commits into from
Jun 8, 2021

Conversation

jeremyk-91
Copy link
Contributor

Goals (and why):

  • Upgrade gradle-baseline.
  • Introduce source retention for immutables style annotations: this should help to eliminate compilation warnings for consumers that do not directly depend on immutables.

Implementation Description (bullets):

  • Switch to using meta-annotations that can be configured to be only retained in source jars.

Testing (What was existing testing like? What have you done to improve it?):

  • Does it build? I think other than that the existing tests passing should be good enough evidence.

Concerns (what feedback would you like?):

Where should we start reviewing?: ImmutablesStyles.java

Priority (whenever / two weeks / yesterday): this week please: not good to get behind on dependencies.

@jeremyk-91 jeremyk-91 requested a review from anjor June 7, 2021 18:39
@changelog-app
Copy link

changelog-app bot commented Jun 7, 2021

Generate changelog in changelog/@unreleased

Type

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

Description

Immutables style annotations now have source retention.

Check the box to generate changelog(s)

  • Generate changelog entry

@anjor
Copy link
Contributor

anjor commented Jun 8, 2021

[akanekar@akanekar22-mac:atlasdb (jkong/package-visible-immutables)]$ grep -r "\@Value.Style" * atlasdb-client/src/main/java/com/palantir/atlasdb/table/description/TableMetadataStyle.java:@Value.Style(get = {"get*", "is*", "has*"}) atlasdb-commons/src/main/java/com/palantir/common/annotations/ImmutablesStyles.java: @Value.Style(visibility = Value.Style.ImplementationVisibility.PACKAGE) atlasdb-commons/src/main/java/com/palantir/common/annotations/ImmutablesStyles.java: @Value.Style(stagedBuilder = true) atlasdb-commons/src/main/java/com/palantir/common/annotations/ImmutablesStyles.java: @Value.Style(stagedBuilder = false) // Intentional: used for e.g. inner classes of staged builder public classes. atlasdb-commons/src/main/java/com/palantir/common/annotations/ImmutablesStyles.java: @Value.Style(allParameters = true) atlasdb-commons/src/main/java/com/palantir/common/annotations/ImmutablesStyles.java: @Value.Style(attributeBuilderDetection = true) timelock-agent/src/main/java/com/palantir/atlasdb/timelock/paxos/DoNotMatchInnerBuilderStyle.java:@Value.Style(typeInnerBuilder = "DoNotMatch") [akanekar@akanekar22-mac:atlasdb (jkong/package-visible-immutables)]$ grep -r "\@Style" * [akanekar@akanekar22-mac:atlasdb (jkong/package-visible-immutables)]$

looks like you got all of them!

@bulldozer-bot bulldozer-bot bot merged commit 9243896 into develop Jun 8, 2021
@bulldozer-bot bulldozer-bot bot deleted the jkong/package-visible-immutables branch June 8, 2021 11:07
@svc-autorelease
Copy link
Collaborator

Released 0.347.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants