Skip to content

Render steering and emeritus member tables in README.md - #377

Merged
pavelzw merged 2 commits into
mainfrom
render-readme-member-tables
Aug 6, 2026
Merged

Render steering and emeritus member tables in README.md#377
pavelzw merged 2 commits into
mainfrom
render-readme-member-tables

Conversation

@jezdez

@jezdez jezdez commented Apr 16, 2026

Copy link
Copy Markdown
Member

Replaces the {{ steering_members }} and {{ emeritus_members }} placeholders in the governance policy README with actual markdown tables generated from steering.csv and emeritus.csv.

Extends the sync script from #374 with build_table() and two new marker pairs (STEERING-TABLE, EMERITUS-TABLE). Also renames the vote checklist markers to STEERING-CHECKLIST for consistency.

Drift detection and auto-fix are handled by a pre-commit hook that runs pixi run sync_steering_members, replacing the dedicated GitHub Actions workflows.

Stacked on top of #380 for the pixi-based pre-commit infrastructure. Will need a re-target to `main` (and a rebase) once #380 lands.

Comment thread .github/workflows/_sync-steering-members.yml Outdated
@jezdez
jezdez force-pushed the render-readme-member-tables branch 5 times, most recently from 4b438c5 to 9b48b6c Compare April 28, 2026 10:09
@jezdez
jezdez marked this pull request as ready for review April 28, 2026 10:10
@jezdez
jezdez requested a review from a team as a code owner April 28, 2026 10:10
@jezdez
jezdez requested a review from jaimergp April 28, 2026 10:10
Comment thread .github/workflows/check-steering-members.yml Outdated
@jezdez
jezdez marked this pull request as draft April 28, 2026 11:35
@jezdez
jezdez force-pushed the render-readme-member-tables branch from 3632d8c to 2c5480b Compare April 28, 2026 11:40
@jezdez
jezdez changed the base branch from main to rework-pre-commit April 28, 2026 11:40
@jezdez
jezdez force-pushed the rework-pre-commit branch from 0b02025 to 4e59bd1 Compare April 28, 2026 11:43
@jezdez
jezdez force-pushed the render-readme-member-tables branch from 2c5480b to f823e0b Compare April 28, 2026 11:44
jezdez added a commit that referenced this pull request Apr 28, 2026
These three files predate the rebase onto current main. Applies the
new yamlfmt config and adds a targeted zizmor ignore for the
artifact-only workflow_run consumer. Both files will be removed by
PR #377.
@jezdez
jezdez force-pushed the render-readme-member-tables branch 2 times, most recently from 993d728 to b4ec42e Compare April 28, 2026 11:49
@jezdez
jezdez force-pushed the rework-pre-commit branch from 3b71c6c to 279fa59 Compare April 28, 2026 11:55
@jezdez
jezdez force-pushed the render-readme-member-tables branch 2 times, most recently from ce3f0af to 7d94beb Compare April 28, 2026 12:00
Base automatically changed from rework-pre-commit to main July 24, 2026 13:35
@pavelzw

pavelzw commented Jul 24, 2026

Copy link
Copy Markdown
Member

#380 now landed, would be great if we could get this through. with #422 it will also integrate trivially into CI

@jezdez

jezdez commented Jul 25, 2026

Copy link
Copy Markdown
Member Author

@pavelzw Sure, do you have time to rebase it?

@pavelzw

pavelzw commented Jul 25, 2026

Copy link
Copy Markdown
Member

Only after #422

@jaimergp

jaimergp commented Aug 5, 2026

Copy link
Copy Markdown
Member

#422 is in!

@pavelzw
pavelzw force-pushed the render-readme-member-tables branch from 7d94beb to cc5dd6e Compare August 5, 2026 17:23
@pavelzw
pavelzw marked this pull request as ready for review August 5, 2026 17:35

@jaimergp jaimergp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lovely, thanks! It would be amazing if we could also generate the CSV for helios votes but I can work on that separately now that we have this.

@pavelzw
pavelzw merged commit 8a84c11 into main Aug 6, 2026
1 check passed
@pavelzw
pavelzw deleted the render-readme-member-tables branch August 6, 2026 11:50
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.

4 participants