Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge DataProvider migration recipe from MBoegers/migrate-testngtojupiter-rewrite #38

Draft
wants to merge 51 commits into
base: main
Choose a base branch
from

Conversation

Philzen
Copy link
Owner

@Philzen Philzen commented Jun 22, 2024

No description provided.

@Philzen Philzen linked an issue Jun 22, 2024 that may be closed by this pull request
@Philzen Philzen self-assigned this Jun 22, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Some suggestions could not be made:

  • src/main/java/org/philzen/oss/testng/UpdateTestAnnotationToJunit5.java
    • lines 21-21
    • lines 30-30
    • lines 102-102
    • lines 115-115
    • lines 181-181

github-actions[bot]

This comment was marked as spam.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

_

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

_

Copy link

sonarcloud bot commented Jun 22, 2024

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

_

Makes official org.openrewrite.java.testing.* recipes available for
usage in our own master recipe.
Annotation type is J.FieldAccess anytime the annotation is fully qualified.
The code in org.openrewrite.java.testing.junit5.UpdateTestAnnotation
(which this is based on) would build a new annotation instance instead of
modifying the type of the existing annotation. It turns out the latter also
works fine for this recipe's scope.
@Philzen Philzen force-pushed the main branch 30 times, most recently from d7e746f to d606f71 Compare June 24, 2024 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@DataProvider migration
3 participants