Skip to content
This repository was archived by the owner on Jun 2, 2023. It is now read-only.

Add Cucumber-JVM migration module #1

Merged
merged 12 commits into from
Mar 23, 2023
Merged

Add Cucumber-JVM migration module #1

merged 12 commits into from
Mar 23, 2023

Conversation

timtebeek
Copy link
Collaborator

Initial implementation based on export of rewrite-testing-framwork cucumber recipes with permission as both the original author there, and an employee of @moderneinc .

This project aims to help users of cucumber-jvm migrate to newer versions by changing their code to adopt breaking changes.

@timtebeek timtebeek requested a review from mpkorstanje March 12, 2023 15:34
@timtebeek timtebeek self-assigned this Mar 12, 2023
timtebeek added a commit to openrewrite/rewrite-testing-frameworks that referenced this pull request Mar 12, 2023
- uses: actions/checkout@v3
- uses: cucumber/[email protected]
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Note to self: Add this to the secrets.

Copy link
Contributor

@mpkorstanje mpkorstanje left a comment

Choose a reason for hiding this comment

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

I'll try and pass for fixing the problems. Mostly small fry issues.

@mpkorstanje
Copy link
Contributor

@timtebeek I won't get around to attending these changes today. If you happen to have time in the 14 days do feel free.

@timtebeek
Copy link
Collaborator Author

Thanks for reviewing; applied your suggestions just now.

Had the version set for 0.0.1-SNAPSHOT, but maybe 0.1.0-SNAPSHOT would be better; what do you think?

@mpkorstanje
Copy link
Contributor

Yup 0.1.0-SNAPSHOT. First feature release of an experimental module.

timtebeek added a commit to openrewrite/rewrite-testing-frameworks that referenced this pull request Mar 21, 2023
@mpkorstanje mpkorstanje self-assigned this Mar 23, 2023
Copy link
Contributor

@mpkorstanje mpkorstanje left a comment

Choose a reason for hiding this comment

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

Looks good! I'll still have to setup the release environment to make the first release. Cheers!

@mpkorstanje mpkorstanje changed the title Cucumber-JVM migration module Add Cucumber-JVM migration module Mar 23, 2023
@mpkorstanje mpkorstanje merged commit 0d2fe64 into main Mar 23, 2023
@mpkorstanje mpkorstanje deleted the initial_export branch March 23, 2023 22:12
@timtebeek
Copy link
Collaborator Author

Thanks for the merge! Once it's released I'll pick it up on our end to make it available in the platform, and see if there's any projects I can use to try it out.

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

Successfully merging this pull request may close these issues.

2 participants