Skip to content

ci: update the release workflow on main - #1122

Merged
edkerk merged 1 commit into
mainfrom
ci/release-workflow-on-main
Sep 29, 2026
Merged

edkerk merged 1 commit into
mainfrom
ci/release-workflow-on-main

Conversation

@edkerk

@edkerk edkerk commented Sep 29, 2026

Copy link
Copy Markdown
Member

Updates .github/workflows/release.yml on main to the version merged into develop in #1078. No other files change.

workflow_dispatch runs the workflow file of the default branch, so the Release workflow started from the Actions tab uses main's copy. Without this update, that copy would open the release pull request without the notes placeholder and point to a docs/releaseNotes/<version>.md file that no longer exists.

With this version:

  • The release pull request opens with a placeholder for the release notes in its description.
  • check-notes keeps it from merging until the notes are written.
  • On merge, the description (without its HTML comments) is published as the text of the GitHub release, which Zenodo archives with the version.

The file is identical to develop's. It was tested in #1078: the notes check was tried on sample descriptions, validate accepts 2.1.0, and every workflow parses.

workflow_dispatch runs the workflow file of the default branch, so the
Release workflow started from the Actions tab uses main's copy. This brings
it in line with develop (#1078): the release notes are written in the release
pull request's description, which a check requires and the release publishes.
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Model quality report

⚠️ 6 pre-existing finding(s), no regressions vs main. Non-blocking.

Each check name links to its explanation in the testResults README.

Model checks

Duplicate keys (model unloadable) and no growth block the merge; every other row is a non-blocking report.

Check Result Δ vs main
Duplicate !!omap keys 0 new ✅
Growth (biomass producible) 125 new ✅
Reactions with no metabolites 0 new ✅
Model / annotation-table inconsistencies 0 new ✅
Removed reactions or metabolites not deprecated 0 new ✅
Metabolites missing formula 0 new ✅
Metabolites missing charge 0 new ✅
Reaction bound / GPR issues 0 new ✅
Exact-duplicate reaction groups 0 new ✅
Unused metabolites 0 new ✅
Unused genes 0 new ✅
Malformed cross-references 0 new ✅
Cross-refs inconsistent across compartments 3 new ⚠️

MACAW and mass/charge balance

Check Result Δ vs main
Reactions flagged by MACAW dead-end test 2510 -703 ⚠️
Reactions flagged as MACAW duplicates 377 -2 ⚠️
Mass-imbalanced reactions 87 new ⚠️
Charge-imbalanced reactions 234 new ⚠️
Structure vs formula/charge inconsistencies 397 new ⚠️

Model file and metabolic tasks

Check Result
YAML round-trip (cobrapy) pass ✅
YAML round-trip (RAVEN) pass ✅
YAML lint pass ✅
Essential metabolic tasks 57 passed ✅
Verification metabolic tasks 21 passed ✅

MEMOTE

Total score: 63.2% (core subset)  

Section Score Δ vs base
consistency 42.4%
annotation_met 73.0%
annotation_rxn 72.7%
annotation_gene 46.7%
annotation_sbo 81.7%
Per-test scores
Section Test Score
Consistency Stoichiometric Consistency 100.0%
Consistency Mass Balance 0.8%
Consistency Charge Balance 2.1%
Consistency Metabolite Connectivity 0.0%
Consistency Unbounded Flux In Default Medium 100.0%
Annotation - Metabolites Presence of Metabolite Annotation 0.0%
Annotation - Metabolites Metabolite Annotations Per Database 62.3%
Annotation - Metabolites Metabolite Annotation Conformity Per Database 45.8%
Annotation - Metabolites Uniform Metabolite Identifier Namespace 0.0%
Annotation - Reactions Presence of Reaction Annotation 0.0%
Annotation - Reactions Reaction Annotations Per Database 75.9%
Annotation - Reactions Reaction Annotation Conformity Per Database 33.3%
Annotation - Reactions Uniform Reaction Identifier Namespace 0.0%
Annotation - Genes Presence of Gene Annotation 0.0%
Annotation - Genes Gene Annotations Per Database 80.0%
Annotation - Genes Gene Annotation Conformity Per Database 80.0%
Annotation - SBO Terms Metabolite General SBO Presence 0.0%
Annotation - SBO Terms Metabolite SBO:0000247 Presence 0.1%
Annotation - SBO Terms Reaction General SBO Presence 0.0%
Annotation - SBO Terms Metabolic Reaction SBO:0000176 Presence 0.0%
Annotation - SBO Terms Transport Reaction SBO:0000185 Presence 0.7%
Annotation - SBO Terms Exchange Reaction SBO:0000627 Presence 0.0%
Annotation - SBO Terms Demand Reaction SBO:0000628 Presence 100.0%
Annotation - SBO Terms Sink Reactions SBO:0000632 Presence 100.0%
Annotation - SBO Terms Gene General SBO Presence 0.0%
Annotation - SBO Terms Gene SBO:0000243 Presence 0.0%
Annotation - SBO Terms Biomass Reactions SBO:0000629 Presence 0.0%

Full suite: 64.2%   · from the last /run memote.

The score above is the fast core subset. Comment /run memote to run the full suite on this pull request; the score updates here when it finishes.

Gene essentiality (Hart 2015)

Not run automatically (it takes hours). Comment /run gene-essentiality to run it on this pull request; the result posts as its own comment.

❌ = a count rose vs the target branch (regression) · ⚠️ = a pre-existing non-zero finding (non-blocking) · ⏳ = still running. Counts link to the CSV listing the exact entries.

Full workflow run · this comment is edited as results come in

@edkerk
edkerk merged commit 7d67835 into main Sep 29, 2026
5 of 6 checks passed
@edkerk
edkerk deleted the ci/release-workflow-on-main branch September 29, 2026 16:13
@edkerk edkerk mentioned this pull request Sep 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant