Skip to content

chore: remove dead files left over from the monorepo merge - #2027

Merged
spydon merged 2 commits into
mainfrom
lukasklingsbo/sdk-2974-remove-dead-files-left-over-from-the
Nov 11, 2026
Merged

spydon merged 2 commits into
mainfrom
lukasklingsbo/sdk-2974-remove-dead-files-left-over-from-the

Conversation

@spydon

@spydon spydon commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

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/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.

Linear: SDK-2974

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain 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

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.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • 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.

@spydon
spydon merged commit 408b5f1 into main Nov 11, 2026
52 checks passed
@spydon
spydon deleted the lukasklingsbo/sdk-2974-remove-dead-files-left-over-from-the branch November 11, 2026 10:04
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