Skip to content

docs(django-spanner, common-protos): centralize CONTRIBUTING.rst pointers - #18041

Merged
hebaalazzeh merged 1 commit into
mainfrom
feat/centralize-contributing-deferred
Aug 20, 2026
Merged

docs(django-spanner, common-protos): centralize CONTRIBUTING.rst pointers#18041
hebaalazzeh merged 1 commit into
mainfrom
feat/centralize-contributing-deferred

Conversation

@chalmerlowe

@chalmerlowe chalmerlowe commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

This PR centralizes the contributing guidelines for django-google-spanner and googleapis-common-protos, redirecting to the root CONTRIBUTING.rst.

These packages were separated from the main batch (see PR #17642) because they were failing import-profiler checks due to pre-existing environment/configuration issues, independent of this change.

It appears that those issues have been resolved.

Blocked by PR #17642

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request replaces individual markdown contribution guides with package-specific RestructuredText (RST) files pointing to the centralized contributing guide at the repository root of the google-cloud-python monorepo. I have no feedback to provide as there are no review comments.

@chalmerlowe
chalmerlowe force-pushed the feat/centralize-contributing-deferred branch from 7dcb45c to 8189812 Compare August 11, 2026 08:56
@chalmerlowe chalmerlowe changed the title feat: centralize contributing for django-google-spanner and googleapis-common-protos docs(django-spanner, common-protos): centralize CONTRIBUTING.rst pointers Aug 19, 2026
@chalmerlowe
chalmerlowe marked this pull request as ready for review August 19, 2026 13:48
@chalmerlowe
chalmerlowe requested review from a team as code owners August 19, 2026 13:48
chalmerlowe added a commit that referenced this pull request Aug 20, 2026
### Problem
1. Each package has a `CONTRIBUTING.rst` file and each file contained
references to supported Python runtimes and specific dependency
versions, etc.
2. The references to supported runtimes and needed dependencies needed
to be updated every time we changed the install environment, which was
unnecessary toil.

### Solution
1. Package-level `CONTRIBUTING.rst` files in handwritten libraries
simply point to a centralized file in the repository root.
2. Removed all references from both the package-level mini-files and the
repository's centralized file that used to spell out which Python
runtime versions apply to this package. Instead we now point users to
the `noxfile.py`, `setup.py`, and `pyproject.toml` as sources of truth
for runtimes and other dependencies.

### Out of Scope/Future work
1. Update GAPIC templates to point at the central `CONTRIBUTING.rst`
file.
2. Failing packages that are being processed independently: 
   * #18041
   * #18142
   * #18157

@hebaalazzeh hebaalazzeh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hebaalazzeh
hebaalazzeh merged commit 2b056ab into main Aug 20, 2026
69 checks passed
@hebaalazzeh
hebaalazzeh deleted the feat/centralize-contributing-deferred branch August 20, 2026 15:25
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.

2 participants