Skip to content

Mark an initial set of recipes as usable for JavaScript#6485

Merged
timtebeek merged 4 commits intomainfrom
mark-an-initial-few-recipes-for-npm
Jan 7, 2026
Merged

Mark an initial set of recipes as usable for JavaScript#6485
timtebeek merged 4 commits intomainfrom
mark-an-initial-few-recipes-for-npm

Conversation

@timtebeek
Copy link
Copy Markdown
Member

Couple quick questions from my side:

  • how do we best validate that recipes work?
  • any others would we want to add already?

@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Jan 6, 2026
@timtebeek timtebeek marked this pull request as ready for review January 7, 2026 08:47
@timtebeek timtebeek changed the title Mark an initial set of recipes as usable for npm ecosystem Mark an initial set of recipes as usable for JavaScript Jan 7, 2026
@timtebeek timtebeek added recipe Requested Recipe javascript labels Jan 7, 2026
Adds an integration test, which tests that a JS recipe can call a Java recipe loaded via marketplace.
@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite Jan 7, 2026
@timtebeek timtebeek merged commit bf0f0f6 into main Jan 7, 2026
2 checks passed
@timtebeek timtebeek deleted the mark-an-initial-few-recipes-for-npm branch January 7, 2026 14:59
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Jan 7, 2026
timtebeek added a commit that referenced this pull request Apr 28, 2026
…7510)

These cross-ecosystem rows were inadvertently removed in #6991 when
recipes.csv was regenerated by `MavenRecipeMarketplaceGenerator`, which
overwrites the file from a pure classpath scan and does not preserve
the manually-added rows that advertise these Java recipes to the
JavaScript (#6485), Python (#6593), and C# (#6678) ecosystems.

Restores `ChangeMethodName`, `ChangePackage`, `ChangeType`,
`DeleteMethodArgument`, `ReorderMethodArguments`, `search.FindMethods`,
and `search.FindTypes` for each of those three ecosystems, derived from
the current Java rows so they stay in sync with the latest option and
data table metadata.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript recipe Requested Recipe

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants