Skip to content

enhance matching#193

Open
aaronhurt wants to merge 1 commit intohelm:mainfrom
aaronhurt:ahurt/enhance-matching
Open

enhance matching#193
aaronhurt wants to merge 1 commit intohelm:mainfrom
aaronhurt:ahurt/enhance-matching

Conversation

@aaronhurt
Copy link
Copy Markdown

This commit reworks matching. The new matcher ignores spacing and order differences from Helm. The format of the Map.yaml file has been simplified to reflect these changes. The README.md has been updated, and a new converter tool has been provided. The current Map.yaml in this PR was generated from the previous Map.yaml via the converter.

This PR is breaking in it's current state, it will only accept the new Map.yaml format, but as stated previously a converter has been provided.

@aaronhurt aaronhurt force-pushed the ahurt/enhance-matching branch from fd9f581 to f444867 Compare March 13, 2025 14:11
@aaronhurt aaronhurt force-pushed the ahurt/enhance-matching branch 2 times, most recently from 331aa41 to 31e9186 Compare April 7, 2025 20:21
@robertsirc
Copy link
Copy Markdown
Member

Hello, thank you for your PR, this branch has some conflicts could you please correct this and we can take a look.

@aaronhurt aaronhurt force-pushed the ahurt/enhance-matching branch 2 times, most recently from 46e0694 to a6ac4d3 Compare April 6, 2026 20:34
This commit reworks matching. The new matcher ignores spacing and order
differences from Helm. The format of the `Map.yaml` file has been
simplified to reflect these changes. The README.md has been updated,
and a new converter tool has been provided. The current `Map.yaml` in
this PR was generated from the previous `Map.yaml` via the converter.

Signed-off-by: Aaron Hurt <aaron.hurt@bayer.com>
@aaronhurt aaronhurt force-pushed the ahurt/enhance-matching branch from a6ac4d3 to 7053b02 Compare April 6, 2026 20:48
@aaronhurt
Copy link
Copy Markdown
Author

Hi @robertsirc, sorry for the delay here. I've rebased my fork off the current main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants