Skip to content

ci: Remove volatile metadata from generated models header#718

Merged
vdusek merged 2 commits intomasterfrom
fix/remove-volatile-metadata-from-generated-models
Apr 13, 2026
Merged

ci: Remove volatile metadata from generated models header#718
vdusek merged 2 commits intomasterfrom
fix/remove-volatile-metadata-from-generated-models

Conversation

@vdusek
Copy link
Copy Markdown
Contributor

@vdusek vdusek commented Apr 13, 2026

Summary

  • Disable timestamp and filename in the datamodel-codegen header via pyproject.toml config
  • Prevents spurious PRs when the OpenAPI spec hasn't changed but regeneration produces a diff due to volatile metadata (timestamp, source URL).

Test plan

  • Regenerated models locally — header is now a single static line # generated by datamodel-codegen
  • CI passes

🤖 Generated with Claude Code

Disable timestamp and filename in the datamodel-codegen header so that
regenerating from an unchanged OpenAPI spec produces identical output,
preventing spurious PRs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vdusek vdusek added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Apr 13, 2026
@vdusek vdusek self-assigned this Apr 13, 2026
@vdusek vdusek changed the title fix: remove volatile metadata from generated models header ci: Remove volatile metadata from generated models header Apr 13, 2026
@github-actions github-actions bot added this to the 138th sprint - Tooling team milestone Apr 13, 2026
@vdusek vdusek requested a review from Pijukatel April 13, 2026 08:02
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.24%. Comparing base (8a0e8e1) to head (67ddd05).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #718      +/-   ##
==========================================
- Coverage   95.34%   95.24%   -0.11%     
==========================================
  Files          45       45              
  Lines        4812     4812              
==========================================
- Hits         4588     4583       -5     
- Misses        224      229       +5     
Flag Coverage Δ
integration 95.24% <ø> (-0.11%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek merged commit 261e2a3 into master Apr 13, 2026
24 checks passed
@vdusek vdusek deleted the fix/remove-volatile-metadata-from-generated-models branch April 13, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants