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

MigrateApiModelToSchema #17

Merged
merged 3 commits into from
Oct 6, 2024
Merged

MigrateApiModelToSchema #17

merged 3 commits into from
Oct 6, 2024

Conversation

OlegZavrazhin
Copy link
Contributor

MigrateApiModelToSchema, position removal from ApiModelProperty

What's changed?

I added MigrateApiModelToSchema. Also adjusted position attribute removal from ApiPropertyModel

What's your motivation?

I encountered difficulties during migration applying rewrite-openapi

Anything in particular you'd like reviewers to focus on?

There may be something that doesn't match your guidelines, please let me know if you need to change anything.

Have you considered any alternatives or workarounds?

I'd like to know if there is possibility to create and run your own recipes.

Checklist

  • I've added unit tests to cover both positive and negative cases
  • I've read and applied the recipe conventions and best practices
  • I've used the IntelliJ IDEA auto-formatter on affected files

MigrateApiModelToSchema, position removal from ApiModelProperty
@timtebeek timtebeek self-requested a review October 1, 2024 08:19
@timtebeek timtebeek added the enhancement New feature or request label Oct 1, 2024
@OlegZavrazhin
Copy link
Contributor Author

@timtebeek hello. Are any changes to the code required?

Copy link
Contributor

@timtebeek timtebeek left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @OlegZavrazhin ! I'd been traveling hence the delay in review and merge. This should make it into Wednesday's release. :)

@timtebeek
Copy link
Contributor

timtebeek commented Oct 6, 2024

I'd like to know if there is possibility to create and run your own recipes.

Absolutely! You can follow along to this workshop to get started with your own custom in house recipe library.

Anything you think could benefit others we'd love to adopt & support going forward.

@timtebeek timtebeek merged commit 178666d into openrewrite:main Oct 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants