daviddallakyan2005 opened a new pull request, #3802:
URL: https://github.com/apache/iceberg-python/pull/3802

   Closes #1191
   
   # Rationale for this change
   
   `InspectTable` in `pyiceberg/table/inspect.py` had no docstrings. This ports 
the review-approved text from #3457 (RyukR1; approved by Fokko, rambleraptor, 
and ebyhr) onto current `main`.
   
   #3457 was CI-green and approved, then stale-closed. #3555 was a later 
attempt that closed unreviewed; there is no open PR.
   
   Two docstring-only adjustments versus #3457, because `main` moved:
   - document the `key_metadata` column on `all_manifests` / 
`_get_all_manifests_schema`
   - spacing on ``spec_id`` in `partitions`
   
   No code changes. RyukR1 is credited as co-author.
   
   ## Are these changes tested?
   
   - `make lint` — all hooks pass, including pydocstyle and mypy
   - `make test` — 3929 passed, 3 skipped
   
   ## Are there any user-facing changes?
   
   No behavior or API changes. Docstrings only; help text / IDE discoverability 
for the existing inspect methods.
   
   Made with [Cursor](https://cursor.com)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to