You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
chore: remove dead files left over from the monorepo merge - #2027
Removes files that came over from the old per-package repositories and no longer do anything.
src/storage/MAINTAINERS.md: the old storage-py maintainer list. .github/CODEOWNERS covers ownership now.
src/functions/conftest.py: implemented an incremental pytest marker that no test uses. The same unused hooks are also removed from src/auth/tests/conftest.py, which keeps only the shared fake instance.
.coveragerc: pytest runs from src/<pkg>, so the root file was never read, and it omitted docs/* and examples/*, which no longer exist.
src/postgrest/LICENSE: an identical copy of the root LICENSE. pyproject.toml does not reference it, and a local uv build --package postgrest confirmed it is not included in the wheel or the sdist, so nothing shipped changes.
.gitignore: dropped the second copy of the GitHub Python template. The four patterns that only existed in that copy (.python-version, .direnv, .envrc, .cursor/) are moved into the matching sections of the first copy, so the set of ignored files stays the same. The .mypy_cache/, __pycache__/ and tags lines that were repeated at the top are removed, and the inline comment on !*.svg moves to its own line, because gitignore does not support inline comments and the negation never applied.
README.md: the end-of-life example now uses Python 3.12 (October 2028) instead of 3.9, since 3.12 is the minimum supported version.
Testing
pytest and pyrefly check in src/functions pass without the removed conftest.py.
The offline auth tests pass with the trimmed conftest.py.
git ls-files -ci --exclude-standard lists no tracked files ignored after the .gitignore change.
No actionable comments were generated in the recent review.
Summary
Summary by CodeRabbit
Documentation
Updated the Python support policy example to reference Python 3.12’s expected end-of-life date.
Chores
Updated repository ignore rules for local Python version, environment, and editor configuration files.
Changed test-runner behavior: later incremental tests are no longer automatically marked as expected failures when an earlier matching test fails.
The changes remove coverage omission settings and incremental-test failure hooks. They update .gitignore rules and change the README’s Python end-of-life example from Python 3.9 to Python 3.12. They also delete the PostgREST license file and the storage maintainers page.
Possibly related PRs
supabase/supabase-py#1370: Changes the documented Python support policy and removes Python 3.9 from CI.
supabase/supabase-py#2002: Raises the minimum supported Python version to 3.12 across metadata, CI, and documentation.
supabase/supabase-py#1963: Removes mypy tooling that is also referenced by ignore rules changed here.
Change: Other
Merge Risk: Minimal · up to 76884
This is a cleanup of leftover repository files and configuration with no runtime behavior change. No actionable merge-blocking risk is identified.
- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 159f7e23-709c-42a7-9ad9-fe7361e3daf0
Commits
Reviewing files that changed from the base of the PR and between 29843bd and 76884b1.
Files selected for processing: 7
.coveragerc
.gitignore
README.md
src/auth/tests/conftest.py
src/functions/conftest.py
src/postgrest/LICENSE
src/storage/MAINTAINERS.md
Files with no reviewable changes: 5
src/postgrest/LICENSE
src/storage/MAINTAINERS.md
.coveragerc
src/functions/conftest.py
src/auth/tests/conftest.py
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 5 remain after this review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes files that came over from the old per-package repositories and no longer do anything.
src/storage/MAINTAINERS.md: the old storage-py maintainer list..github/CODEOWNERScovers ownership now.src/functions/conftest.py: implemented anincrementalpytest marker that no test uses. The same unused hooks are also removed fromsrc/auth/tests/conftest.py, which keeps only the sharedfakeinstance..coveragerc: pytest runs fromsrc/<pkg>, so the root file was never read, and it omitteddocs/*andexamples/*, which no longer exist.src/postgrest/LICENSE: an identical copy of the rootLICENSE.pyproject.tomldoes not reference it, and a localuv build --package postgrestconfirmed it is not included in the wheel or the sdist, so nothing shipped changes..gitignore: dropped the second copy of the GitHub Python template. The four patterns that only existed in that copy (.python-version,.direnv,.envrc,.cursor/) are moved into the matching sections of the first copy, so the set of ignored files stays the same. The.mypy_cache/,__pycache__/andtagslines that were repeated at the top are removed, and the inline comment on!*.svgmoves to its own line, because gitignore does not support inline comments and the negation never applied.README.md: the end-of-life example now uses Python 3.12 (October 2028) instead of 3.9, since 3.12 is the minimum supported version.Testing
pytestandpyrefly checkinsrc/functionspass without the removedconftest.py.conftest.py.git ls-files -ci --exclude-standardlists no tracked files ignored after the.gitignorechange.Linear: SDK-2974