Skip to content

Check if XML file uses NS URI#5496

Merged
sambsnyd merged 6 commits intomainfrom
boeg/xml-has-not-ns-uri
May 26, 2025
Merged

Check if XML file uses NS URI#5496
sambsnyd merged 6 commits intomainfrom
boeg/xml-has-not-ns-uri

Conversation

@MBoegers
Copy link
Contributor

What's changed?

Add a search recipe that finds XML root nodes that (or a child) use a namespace uri.

What's your motivation?

enable openrewrite/rewrite-migrate-java#741

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

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context

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

@MBoegers MBoegers self-assigned this May 26, 2025
@MBoegers MBoegers added the enhancement New feature or request label May 26, 2025
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite May 26, 2025
MBoegers and others added 2 commits May 26, 2025 14:21
…espaceUri.java

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this from In Progress to Ready to Review in OpenRewrite May 26, 2025
@sambsnyd sambsnyd merged commit b9bdbdc into main May 26, 2025
2 checks passed
@sambsnyd sambsnyd deleted the boeg/xml-has-not-ns-uri branch May 26, 2025 18:48
@github-project-automation github-project-automation bot moved this from Ready to Review to Done in OpenRewrite May 26, 2025
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.

4 participants