Skip to content

fix(openapi): prevent make dev from rewriting api.yaml version on event#8334

Open
axsseldz wants to merge 1 commit intomarimo-team:mainfrom
axsseldz:fix/make-dev-openapi-api-yaml-version-drift
Open

fix(openapi): prevent make dev from rewriting api.yaml version on event#8334
axsseldz wants to merge 1 commit intomarimo-team:mainfrom
axsseldz:fix/make-dev-openapi-api-yaml-version-drift

Conversation

@axsseldz
Copy link
Contributor

…ry run

📝 Summary

Running make dev rewrites packages/openapi/api.yaml every time, even without API/schema changes.

Closes #8333.

🔍 Description of Changes

  • info.version is updated (example: 0.19.10 -> 0.19.11)

This creates repeated noise in git diff during normal local development.

📋 Checklist

  • I have read the contributor guidelines.
  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Tests have been added for the changes made.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Pull request title is a good summary of the changes - it will be used in the release notes.

CC: @mscolnick @nojaf

@vercel
Copy link

vercel bot commented Feb 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Feb 16, 2026 10:50pm

Request Review

@github-actions github-actions bot added the bash-focus Area to focus on during release bug bash label Feb 16, 2026
@nojaf
Copy link
Contributor

nojaf commented Feb 17, 2026

@axsseldz can you investigate when this version bumps?
Can we have a CI check for this? Or some pre-commit hook that only hooks in when the relevant file changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bash-focus Area to focus on during release bug bash

Projects

None yet

Development

Successfully merging this pull request may close these issues.

api.yaml (info.version) on every run

2 participants