Skip to content

fix: add epic_craterlake (with bhcal) to step export#61

Open
wdconinc wants to merge 1 commit into
perf-geocad-shape-creationfrom
step-export-bhcal
Open

fix: add epic_craterlake (with bhcal) to step export#61
wdconinc wants to merge 1 commit into
perf-geocad-shape-creationfrom
step-export-bhcal

Conversation

@wdconinc
Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

This PR reenables the BHcal in the full STEP export, since we want it to work and this is how we keep it working :-)

What is the urgency of this PR?

  • High (please describe reason below)
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue: missing BHcal in testing)
  • New feature (issue #__)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

Copilot AI review requested due to automatic review settings May 22, 2026 22:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the CI STEP-export workflow matrix to include the full epic_craterlake detector configuration (with BHCal enabled), restoring coverage for BHCal presence in full STEP export testing.

Changes:

  • Add epic_craterlake to the detector_configs list passed to the reusable convert-to-step.yml workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 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