Skip to content

Conversation

Picnic-DevPla-Bot
Copy link
Contributor

@Picnic-DevPla-Bot Picnic-DevPla-Bot commented Apr 16, 2025

This PR contains the following updates:

Package Type Update Change
OpenRewrite Templating minor 1.25.1 -> 1.26.1
OpenRewrite Templating compile minor 1.25.1 -> 1.26.1

Release Notes

openrewrite/rewrite-templating (OpenRewrite Templating)

v1.26.1

Compare Source

What's Changed

Full Changelog: openrewrite/rewrite-templating@v1.26.0...v1.26.1

v1.26.0

Compare Source

What's Changed
New Contributors

Full Changelog: openrewrite/rewrite-templating@v1.25.1...v1.26.0


  • If you want to rebase/retry this PR, check this box

@Picnic-DevPla-Bot
Copy link
Contributor Author

Picnic-DevPla-Bot commented Apr 16, 2025

Suggested commit message:

Upgrade OpenRewrite Templating 1.25.1 -> 1.26.1 (#1644)

See:
- https://github.com/openrewrite/rewrite-templating/releases/tag/v1.26.0
- https://github.com/openrewrite/rewrite-templating/releases/tag/v1.26.1
- https://github.com/openrewrite/rewrite-templating/compare/v1.25.1...v1.26.1

Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

2 similar comments
Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@timtebeek
Copy link
Contributor

[INFO] -------------------------------------------------------------
Warning:  COMPILATION WARNING : 
[INFO] -------------------------------------------------------------
Warning:  /Users/runner/work/error-prone-support/error-prone-support/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/AssertJRules.java:[182,37] unchecked generic array creation for varargs parameter of type E[]
Warning:  /Users/runner/work/error-prone-support/error-prone-support/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/AssertJRules.java:[183,47] unchecked generic array creation for varargs parameter of type E[]
Warning:  /Users/runner/work/error-prone-support/error-prone-support/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/AssertJRules.java:[184,34] unchecked generic array creation for varargs parameter of type E[]

@SuppressWarnings("unchecked")
void before(
ObjectEnumerableAssert<?, E> enumAssert,
@Matches(IsEmpty.class) Iterable<? extends E> wellTypedIterable,
@Matches(IsEmpty.class) Iterable<?> arbitrarilyTypedIterable) {
Refaster.anyOf(
enumAssert.containsExactlyElementsOf(wellTypedIterable),
enumAssert.containsExactlyInAnyOrderElementsOf(wellTypedIterable),
enumAssert.hasSameElementsAs(wellTypedIterable),
enumAssert.hasSameSizeAs(arbitrarilyTypedIterable),
enumAssert.isSubsetOf(wellTypedIterable),
enumAssert.containsExactly(),
enumAssert.containsExactlyInAnyOrder(),
enumAssert.containsOnly(),
enumAssert.isSubsetOf());
}

@Stephan202
Copy link
Member

Interestingly, this only happens when rewrite-templating is enabled. There seem to be two types of warning:

  1. Mostly it's unchecked generic array creation for varargs parameter of type X[].
  2. For this line it says:
    [WARNING] /path/to/error-prone-support/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/MockitoRules.java:[73,85] unchecked cast
      required: T
      found:    java.lang.Object
    

That second warning looks wrong, and seems to hint at generic type erasure (?). I also see that ~each warning is emitted twice. 🤔

I may not have time for a deep-dive in the coming days; @timtebeek do you guys plan to have a look?

@timtebeek
Copy link
Contributor

You're fine to skip this particular version; @fmodesto is still working through his branch of this project, which is likely to result in another release: https://github.com/fmodesto/error-prone-support/tree/openrewrite

Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

1 similar comment
Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@timtebeek
Copy link
Contributor

@Picnic-DevPla-Bot Picnic-DevPla-Bot changed the title Upgrade OpenRewrite Templating 1.25.1 -> 1.26.0 Upgrade OpenRewrite Templating 1.25.1 -> 1.26.1 Apr 18, 2025
@Picnic-DevPla-Bot Picnic-DevPla-Bot force-pushed the renovate/version.rewrite-templating branch from c52c18e to 80c5c4c Compare April 18, 2025 12:32
Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@Stephan202 Stephan202 added this to the 0.23.0 milestone Apr 18, 2025
Copy link
Member

@Stephan202 Stephan202 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Way more recipes are generated now 🚀. Based on the very crude find . -path "*target/classes/*Recipes*Recipe*class" | wc -l, we went from 896 to 1682 🚀 🚀 🚀 📈

@timtebeek
Copy link
Contributor

That's great! I believe there were still some issues with matching the new generic recipes, but the interface has likely stabilized already. We'll keep an eye on this, thanks!

Copy link
Contributor

@mohamedsamehsalah mohamedsamehsalah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔢

@rickie rickie force-pushed the renovate/version.rewrite-templating branch from 80c5c4c to 2f8ec5c Compare April 19, 2025 19:38
Copy link

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

Copy link

@rickie rickie merged commit 5771e6f into master Apr 19, 2025
17 checks passed
@rickie rickie deleted the renovate/version.rewrite-templating branch April 19, 2025 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants