Skip to content

[Release] Make CI gates explicit and document the migration - #54

Merged
rhorvath02 merged 7 commits into
review/boblib-contract-testsfrom
review/boblib-release-gates
Oct 25, 2026
Merged

rhorvath02 merged 7 commits into
review/boblib-contract-testsfrom
review/boblib-release-gates

Conversation

@rhorvath02

@rhorvath02 rhorvath02 commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Purpose

Make release failures attributable to a specific check and document the public API migration.

Approach

Separate translation, initialization, physics, signal, FMU and component checks into named workflow steps and retain their reports. Add mirrored-fixture profiling to CI and update the user guide, migration notes and release-readiness documentation.

Validation

Every workflow target and referenced file was checked. Complete numerical regression and rendered-diagram acceptance remain release gates; a green default CI run does not certify a release.

@rhorvath02
rhorvath02 force-pushed the review/boblib-release-gates branch from d951b53 to 379e18d Compare October 22, 2026 08:44

@raorjun raorjun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PR CI still does not run most of the new Python tests in this stack. The release gate that does run them is manual and has not run on any review branch. The migration guide gives no modifier map, and BobSim drops unknown keys with no error.

Comment thread makefile
Comment thread .github/workflows/release-gate.yml
Comment thread .github/workflows/ci.yml
Comment thread MIGRATION.md
@raorjun

raorjun commented Oct 23, 2026

Copy link
Copy Markdown
Member

Fixed in three commits: e59f38f runs the release gate on v* tag pushes, and the README now says so. 348aad5 moves the mirrored-fixture profile into its own "OpenModelica Profiles" job with its own 60-minute timeout, so the smoke job has headroom again. 2c6d419 adds an old-to-new map to MIGRATION.md: chassis and axle classes, the template bases, and 17 vcu.* keys that move to driver.*. It also notes that BobSim silently skips unknown keys. I left the chassis stabilizer modifiers out because I could not confirm one new location for every axle type. actionlint is clean. PR CI still does not run make test-additional.

@rhorvath02
rhorvath02 merged commit 155d8ca into main Oct 25, 2026
4 checks passed
@rhorvath02
rhorvath02 deleted the review/boblib-release-gates branch October 25, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-assisted Developed with AI agent assistance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants