Skip to content

Remove redundant Python script formatting from README generation#1879

Open
RobMulla wants to merge 5 commits intomainfrom
fix-downstream-readme
Open

Remove redundant Python script formatting from README generation#1879
RobMulla wants to merge 5 commits intomainfrom
fix-downstream-readme

Conversation

@RobMulla
Copy link
Collaborator

@RobMulla RobMulla commented Mar 6, 2026

Now that generate_support_matrices.sh natively injects fully-formed UI format strings directly into the upstream CSV deliverables, we no longer need downstream Python scripts attempting to intercept and reshape the statuses.

Changes

  • Removed: python3 scripts/update_all_matrices.py from the Buildkite commit wrapper.
  • Removed: python scripts/combine_matrices.py from the GitHub Action.
  • Simplified: update_readme.py now points directly to the core model_support_matrix.csv.
  • Refactored: merge_metrics() and generate_markdown_table() no longer blindly try to scrape emojis or reshape fallback values, relying exclusively on the upstream's single source of truth.

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

Description

Start with a short description of what the PR does and how this is a change from
the past.

The rest of the description includes relevant details and context, examples:

  • why is this change being made,
  • the problem being solved and any relevant context,
  • why this is a good solution,
  • some information about the specific implementation,
  • shortcomings of the solution and possible future improvements.

If the change fixes a Github issue, please include a link, e.g.,:
FIXES: #123456

Tests

Please describe how you tested this change, and include any instructions and/or
commands to reproduce.

Checklist

Before submitting this PR, please make sure:

  • I have performed a self-review of my code.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have made or will make corresponding changes to any relevant documentation.

@RobMulla RobMulla force-pushed the fix-downstream-readme branch 3 times, most recently from 73cc3d1 to a9cfdd0 Compare March 11, 2026 18:38
Signed-off-by: Rob Mulla <rob.mulla@gmail.com>
Signed-off-by: Rob Mulla <rob.mulla@gmail.com>
Signed-off-by: Rob Mulla <rob.mulla@gmail.com>
Signed-off-by: Rob Mulla <rob.mulla@gmail.com>
Signed-off-by: Rob Mulla <rob.mulla@gmail.com>
@RobMulla RobMulla force-pushed the fix-downstream-readme branch from a9cfdd0 to a6e60e4 Compare March 11, 2026 18:41
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