Skip to content

chore(sparkline-basic): altair impl docstring Quality 90/100#5655

Merged
MarkusNeusinger merged 1 commit intomainfrom
chore/sparkline-basic-altair-impl-header
May 2, 2026
Merged

chore(sparkline-basic): altair impl docstring Quality 90/100#5655
MarkusNeusinger merged 1 commit intomainfrom
chore/sparkline-basic-altair-impl-header

Conversation

@MarkusNeusinger
Copy link
Copy Markdown
Owner

Summary

Follow-up to #5653 / #5654. The /regen flow blanked the Quality: line in the docstring header — it now reads Quality: /100, but should match the metadata's quality_score: 90 populated in #5654.

-Quality: /100 | Updated: 2026-05-02
+Quality: 90/100 | Updated: 2026-05-02

The regen.md template (PR #5650) is being updated in parallel to require {SCORE}/100, not blank, so future regens won't drop the score.

Test plan

  • CI green
  • header matches metadata quality_score: 90

Copilot AI review requested due to automatic review settings May 2, 2026 21:59
The /regen flow blanked the Quality field in the docstring header
(Quality: /100). Score should match the metadata's quality_score (90,
populated in #5654). regen.md template (PR #5650) updated in parallel
to require {SCORE}/100, not blank.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a regenerated Altair implementation header for sparkline-basic so the docstring Quality: line is populated again and matches the implementation metadata (quality_score: 90).

Changes:

  • Update plots/sparkline-basic/implementations/python/altair.py docstring header from Quality: /100 to Quality: 90/100.

@MarkusNeusinger MarkusNeusinger merged commit 312f09e into main May 2, 2026
11 checks passed
@MarkusNeusinger MarkusNeusinger deleted the chore/sparkline-basic-altair-impl-header branch May 2, 2026 22:01
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