Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

The cache restore step was positioned after the download step, causing the download to execute on every run and potentially being overwritten by the restored cache.

Changes:

  • Reordered workflow steps: cache restore now precedes download
  • Download only executes on cache miss when ./src/GRCh37.tar.gz doesn't exist
  • Cache save occurs automatically at job completion

Before:

- name: Download GRCh37.tar.gz if not present
  # Always runs, then gets overwritten by cache
- name: Cache src directory
  # Restores after download

After:

- name: Cache src directory
  # Restores first if available
- name: Download GRCh37.tar.gz if not present
  # Only runs on cache miss

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mdbarnesUCSD <22506723+mdbarnesUCSD@users.noreply.github.com>
Copilot AI changed the title [WIP] Update CI migration based on feedback on #241 Fix cache ordering in CI workflow to avoid redundant downloads Jan 23, 2026
Copilot AI requested a review from mdbarnesUCSD January 23, 2026 20:30
@mdbarnesUCSD mdbarnesUCSD merged commit 5a2cab3 into ci-changes Jan 23, 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.

2 participants