Skip to content

CI: Check DVC isexec metadata in "Style Checks" workflow#4615

Merged
seisman merged 2 commits intomainfrom
codex/check-dvc-isexec-in-style-checks
May 2, 2026
Merged

CI: Check DVC isexec metadata in "Style Checks" workflow#4615
seisman merged 2 commits intomainfrom
codex/check-dvc-isexec-in-style-checks

Conversation

@seisman
Copy link
Copy Markdown
Member

@seisman seisman commented May 1, 2026

Summary

  • add a style-check workflow step that scans baseline .png.dvc files for isexec: true
  • fail the workflow if any baseline DVC metadata marks a PNG as executable
  • avoid pulling baseline images from DVC because the metadata check is sufficient
  • Fix permission issue in two baseline images

@seisman seisman changed the title Check DVC isexec metadata in style checks CI: Check DVC isexec metadata in style checks May 1, 2026
@seisman seisman changed the title CI: Check DVC isexec metadata in style checks CI: Check DVC isexec metadata in "Style Checks" workflow May 1, 2026
@seisman seisman added the maintenance Boring but important stuff for the core devs label May 1, 2026
@seisman seisman added this to the 0.19.0 milestone May 1, 2026
@seisman seisman added the needs review This PR has higher priority and needs review. label May 1, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Summary of changed images

This is an auto-generated report of images that have changed on the DVC remote

Status Path

Image diff(s)

Details

Added images

Modified images

Path Old New

Report last updated at commit f13a868

@seisman seisman removed the needs review This PR has higher priority and needs review. label May 2, 2026
@seisman seisman merged commit 400aac0 into main May 2, 2026
21 of 24 checks passed
@seisman seisman deleted the codex/check-dvc-isexec-in-style-checks branch May 2, 2026 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Boring but important stuff for the core devs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants