Skip to content

[superseded] test(backend): run all unit test files in CI#7995

Closed
tianmind-studio wants to merge 1 commit into
BasedHardware:mainfrom
tianmind-studio:codex/windows-testsh-missing-units
Closed

[superseded] test(backend): run all unit test files in CI#7995
tianmind-studio wants to merge 1 commit into
BasedHardware:mainfrom
tianmind-studio:codex/windows-testsh-missing-units

Conversation

@tianmind-studio

@tianmind-studio tianmind-studio commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Superseded by #7890, which has been refreshed onto current main, removes the duplicate test_auth_redirect_uri.py invocation, and now reaches full backend/tests/unit/test_*.py coverage with MISSING_COUNT=0 and DUP_COUNT=0.

Closing this duplicate to keep maintainer review focused on a single runner-cleanup PR.

@tianmind-studio tianmind-studio changed the title test(backend): run all unit test files in CI [superseded] test(backend): run all unit test files in CI Jun 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hey @tianmind-studio 👋

Thank you so much for taking the time to contribute to Omi! We truly appreciate you putting in the effort to submit this pull request.

After careful review, we've decided not to merge this particular PR. Please don't take this personally — we genuinely try to merge as many contributions as possible, but sometimes we have to make tough calls based on:

  • Project standards — Ensuring consistency across the codebase
  • User needs — Making sure changes align with what our users need
  • Code best practices — Maintaining code quality and maintainability
  • Project direction — Keeping aligned with our roadmap and vision

Your contribution is still valuable to us, and we'd love to see you contribute again in the future! If you'd like feedback on how to improve this PR or want to discuss alternative approaches, please don't hesitate to reach out.

Thank you for being part of the Omi community! 💜

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