Skip to content

JavaScript and Python: unique file names for version.txt files#6831

Merged
greg-at-moderne merged 1 commit intomainfrom
greg-rename-version-txt
Feb 26, 2026
Merged

JavaScript and Python: unique file names for version.txt files#6831
greg-at-moderne merged 1 commit intomainfrom
greg-rename-version-txt

Conversation

@greg-at-moderne
Copy link
Copy Markdown
Contributor

@greg-at-moderne greg-at-moderne commented Feb 26, 2026

What's changed?

Rename the special version.txt files baked into rewrite-python and rewrite-javascript JAR files.
New names are unique - different for JS and Python.

What's your motivation?

Prevent JavaScript RPC server from reading the Python one (and vice versa).

The following has been observed in CI of one of the derived projects:

Error output:
      npm error code ETARGET
      npm error notarget No matching version found for @openrewrite/rewrite@8.75.0.dev20260226090025.
      npm error notarget In most cases you or one of your dependencies are requesting
      npm error notarget a package version that doesn't exist.
      npm error A complete log of this run can be found in: /home/runner/.npm/_logs/2026-02-26T12_23_59_857Z-debug-0.log

@greg-at-moderne greg-at-moderne self-assigned this Feb 26, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Feb 26, 2026
@greg-at-moderne greg-at-moderne marked this pull request as ready for review February 26, 2026 13:36
@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite Feb 26, 2026
@greg-at-moderne greg-at-moderne merged commit 8b48c51 into main Feb 26, 2026
1 check passed
@greg-at-moderne greg-at-moderne deleted the greg-rename-version-txt branch February 26, 2026 13:48
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants