Skip to content

CI: Run all scripts and check for modified UUID files - #181

Merged
ubruhin merged 2 commits into
masterfrom
ci-run-scripts
Aug 11, 2026
Merged

CI: Run all scripts and check for modified UUID files#181
ubruhin merged 2 commits into
masterfrom
ci-run-scripts

Conversation

@ubruhin

@ubruhin ubruhin commented Aug 10, 2026

Copy link
Copy Markdown
Member

To have more confidence when reviewing PRs, adding more checks to CI:

  • Actually run all generator scripts (including --3d) to ensure all scripts are working properly
  • Check for diffs in the UUID caches, to ensure all new UUIDs are committed in the PR
  • Clear UUID caches, re-run all generators, and compare the UUID cache file sizes. If any file size has changed, it means that there are stale (unused) UUIDs in the caches which should be removed. This happens regularly if not cleaning up the UUID caches after developing a new script, since UUIDs from old revisions are still there.

Only a few scripts are excluded from these checks:

  • generate_stm_mcu.py: Depends on STM32 device database which is not available in this repo.
  • Some UUID caches contain stale entries already, e.g. because some of the generated packages have been marked as deprecated in the mean time.

Some stale UUIDs are now removed to make the new check pass.

@ubruhin
ubruhin merged commit 4d1d405 into master Aug 11, 2026
8 checks passed
@ubruhin
ubruhin deleted the ci-run-scripts branch August 11, 2026 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant