Skip to content

move eval.sh to tests#2594

Merged
Qubitium merged 1 commit intomainfrom
move-eval-to-tests
Mar 24, 2026
Merged

move eval.sh to tests#2594
Qubitium merged 1 commit intomainfrom
move-eval-to-tests

Conversation

@Qubitium
Copy link
Collaborator

Summary

Describe the bug, fix, or feature clearly and briefly.

What Changed

  • List the main code changes.
  • List any API or behavior changes.
  • List any follow-up work that is intentionally out of scope.

Tests

Every working PR must include at least one new simple, fast, targeted unit test when the change affects behavior, a bug fix, or a regression path.

  • I added a new simple/fast unit test for this change, or documented why that is not applicable.
  • I ran the new targeted test locally before opening this PR.
  • I ran any other directly relevant local tests.

Paste the exact test commands and results here:

Review Requirements

AI-assisted code is welcome.

Every changed file must still be properly reviewed by a human before the PR is opened as ready for review.

We will not accept PRs that are effectively unreviewed AI output. Non-human-reviewed changes often introduce obscure structure, mismatched APIs, project-inconsistent code patterns, or unnecessary monkeypatching instead of a correct fix or clean feature expansion.

  • I personally reviewed every file in this diff.
  • I checked that the code matches existing project structure, APIs, and conventions.
  • I avoided unnecessary monkeypatching and used the project's normal extension points where possible.

Notes

Add any migration notes, risks, compatibility concerns, or reviewer guidance here.

@Qubitium Qubitium merged commit f709e2c into main Mar 24, 2026
5 checks passed
@Qubitium Qubitium deleted the move-eval-to-tests branch March 24, 2026 08:01
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.

1 participant