Skip to content

Support header_rows in ascii.rst writer (swev-id: astropy__astropy-14182) - #122

Open
casey-brooks wants to merge 1 commit into
astropy__astropy-14182from
casey/rst-header-rows
Open

Support header_rows in ascii.rst writer (swev-id: astropy__astropy-14182)#122
casey-brooks wants to merge 1 commit into
astropy__astropy-14182from
casey/rst-header-rows

Conversation

@casey-brooks

Copy link
Copy Markdown

Summary

  • add header_rows plumbing to the RST writer
  • render multi-row headers as grid tables and extend coverage

Testing

  • pytest astropy/io/ascii/tests/test_rst.py::test_write_normal
  • pytest astropy/io/ascii/tests/test_rst_header_rows.py

Refs #113

@casey-brooks
casey-brooks requested a review from a team December 27, 2025 20:22
@casey-brooks

Copy link
Copy Markdown
Author

Test & Lint Summary

  • pytest astropy/io/ascii/tests/test_rst.py::test_write_normal (1 passed)
  • pytest astropy/io/ascii/tests/test_rst_header_rows.py (5 passed)
  • flake8 astropy/io/ascii/rst.py astropy/io/ascii/tests/test_rst_header_rows.py (no issues)

@rowan-stein

Copy link
Copy Markdown
Collaborator

Requesting review.

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good.

@rowan-stein

Copy link
Copy Markdown
Collaborator

Closing to enforce one-open-PR-per-task policy for ID 8. Superseded by PR #126. No merge performed; all work remains in agyn-sandbox.

@rowan-stein rowan-stein reopened this Dec 27, 2025
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.

3 participants