Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[ty] Isolate file-watching test environment#25151

Merged
MichaReiser merged 1 commit into
mainfrom
micha/isolate-ty-file-watching-env
May 14, 2026
Merged

[ty] Isolate file-watching test environment#25151
MichaReiser merged 1 commit into
mainfrom
micha/isolate-ty-file-watching-env

Conversation

@MichaReiser
Copy link
Copy Markdown
Member

@MichaReiser MichaReiser commented May 14, 2026

Summary

The reloading_options_updates_inferred_python_version_diagnostics_when_metadata_is_unchanged file watching test can fail locally when VIRTUAL_ENV is set.

This PR improves the isolation of the file watching tests by removing known environment variables.

@astral-sh-bot astral-sh-bot Bot added the ty Multi-file analysis & type inference label May 14, 2026
@MichaReiser MichaReiser added the testing Related to testing Ruff itself label May 14, 2026
@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented May 14, 2026

Typing conformance results

No changes detected ✅

Current numbers
The percentage of diagnostics emitted that were expected errors held steady at 89.36%. The percentage of expected errors that received a diagnostic held steady at 85.49%. The number of fully passing files held steady at 88/134.

@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented May 14, 2026

Memory usage report

Memory usage unchanged ✅

@MichaReiser MichaReiser force-pushed the micha/isolate-ty-file-watching-env branch from fc4a5d2 to d28b4e3 Compare May 14, 2026 08:37
@MichaReiser MichaReiser marked this pull request as ready for review May 14, 2026 08:38
@astral-sh-bot astral-sh-bot Bot requested a review from charliermarsh May 14, 2026 08:38
@MichaReiser MichaReiser force-pushed the micha/isolate-ty-file-watching-env branch from d28b4e3 to 5995723 Compare May 14, 2026 08:39
@MichaReiser MichaReiser force-pushed the micha/isolate-ty-file-watching-env branch from 5995723 to 76abf39 Compare May 14, 2026 08:41
@MichaReiser MichaReiser enabled auto-merge (squash) May 14, 2026 08:42
@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot Bot commented May 14, 2026

ecosystem-analyzer results

No diagnostic changes detected ✅

Full report with detailed diff (timing results)

@MichaReiser MichaReiser merged commit 828cdb7 into main May 14, 2026
57 checks passed
@MichaReiser MichaReiser deleted the micha/isolate-ty-file-watching-env branch May 14, 2026 08:48
thejchap pushed a commit to thejchap/ruff that referenced this pull request May 23, 2026
## Summary

The
`reloading_options_updates_inferred_python_version_diagnostics_when_metadata_is_unchanged
` file watching test can fail locally when `VIRTUAL_ENV` is set.

This PR improves the isolation of the file watching tests by removing
known environment variables.
anishgirianish pushed a commit to anishgirianish/ruff that referenced this pull request May 28, 2026
## Summary

The
`reloading_options_updates_inferred_python_version_diagnostics_when_metadata_is_unchanged
` file watching test can fail locally when `VIRTUAL_ENV` is set.

This PR improves the isolation of the file watching tests by removing
known environment variables.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Related to testing Ruff itself ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants