Skip to content

fix: Honor trim:false in exact-match normalize - #25

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
VedantMadane:fix/issue-9
Sep 5, 2026
Merged

fix: Honor trim:false in exact-match normalize#25
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
VedantMadane:fix/issue-9

Conversation

@VedantMadane

Copy link
Copy Markdown
Contributor

Summary

Honor trim:false in exact-match normalize

Changes

  • normalize: only collapse whitespace when trim is enabled (default)
  • tests for exact-match trim:false

Fixes #9

- normalize: only collapse whitespace when trim is enabled (default)
- tests for exact-match trim:false

Fixes AgentPostmortem#9

Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Welcome, @VedantMadane, and thanks for your first pull request to Evalgate.

A quick look at CONTRIBUTING.md covers how things are set up and run here.

I'll review this shortly. Ask anything in the thread if something is unclear.

@royalpinto007

Copy link
Copy Markdown
Collaborator

Thanks. Quality-gate is failing on this branch - please check the Gate job log and fix the regression or baseline.

@royalpinto007 royalpinto007 reopened this Sep 5, 2026
@royalpinto007 royalpinto007 reopened this Sep 5, 2026
@royalpinto007 royalpinto007 reopened this Sep 5, 2026
@royalpinto007 royalpinto007 reopened this Sep 5, 2026
@royalpinto007
royalpinto007 merged commit 18a54d3 into AgentPostmortem:main Sep 5, 2026
41 of 46 checks passed
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Merged. Thanks a lot, @VedantMadane.

Genuinely helpful change for Evalgate.

If this project helps you out, a star helps others find it too.

Happy to see more from you whenever you have the time.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

exact-match always collapses internal whitespace, and trim: false does not turn it off

2 participants