Skip to content

FlyBase release check: record the release when the gene id maps are current - #125

Merged
Clare72 merged 1 commit into
mainfrom
flybase-record-current
Nov 10, 2026
Merged

Clare72 merged 1 commit into
mainfrom
flybase-record-current

Conversation

@Clare72

@Clare72 Clare72 commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

The weekly bot commit read "FlyBase n/a": check_flybase_release.py returned early when every gene_id_map.tsv already matched the current FlyBase release, so checks/flybase_release_check.json was never written, and the workflow's commit title falls back to n/a.

  • Maps current and the release is not yet recorded → record it (maps_current: true, no Chado query, no issue) and set changed=true, so the workflow commits it once per release.
  • Release already recorded → nothing to do, as before.
  • The newer-release path is unchanged; its record write now shares a _record helper. --dry-run writes nothing on either path.
  • DESIGN: drop the inaccurate "created on the first FlyBase release check" note.

Tests: the old "same release does nothing" test is replaced by one that checks the release is recorded once (changed=true then changed=false), plus a dry-run test. 131 pass. A real --dry-run against flybase.org reports FB2026_03 current and writes nothing.

The next weekly run should commit checks/flybase_release_check.json for FB2026_03.

🤖 Generated with Claude Code

…urrent

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Clare72
Clare72 merged commit f296e45 into main Nov 10, 2026
1 check passed
@Clare72
Clare72 deleted the flybase-record-current branch November 10, 2026 08:22
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