Skip to content

fix: Correct copied config file name - #255

Open
LKress wants to merge 1 commit into
devfrom
lkress/fix-provenance-config-filename
Open

fix: Correct copied config file name#255
LKress wants to merge 1 commit into
devfrom
lkress/fix-provenance-config-filename

Conversation

@LKress

@LKress LKress commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator
  • This file contains the infomation with which parameters the workflow was run

* This file contains the infomation with which parameters the workflow was run
@LKress
LKress changed the base branch from main to dev August 26, 2026 12:54
@LKress
LKress marked this pull request as ready for review August 26, 2026 12:54
Copilot AI lite review requested due to automatic review settings August 26, 2026 12:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 Snakemake onstart hook to write the run configuration to a more accurate, generic filename in configs/, improving reproducibility and avoiding a misleading stage-specific suffix.

Changes:

  • Rename the saved config output from configs/{timestamp}_pull_resources_config.yaml to configs/{timestamp}_config.yaml.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jonasfreimuth jonasfreimuth left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good catch!

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.

3 participants