Skip to content

Conversation

@kelly-sovacool
Copy link
Member

Changes

  • explicitly pass in the pipeline version to spooker
  • use envsubst to interpolate variables in the config file
  • include the pipeline name & version in the config file for easier spooker usage from snakemake

Issues

for https://github.com/CCBR/ccbrpipeliner/issues/57

PR Checklist

(Strikethrough any points that are not applicable.)

  • This comment contains a description of changes with justifications, with any relevant issues linked.
  • [ ] Update docs if there are any API changes.
  • [ ] Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Guidelines: https://keepachangelog.com/en/1.1.0/

@kopardev kopardev added the ASPEN RepoName label Apr 10, 2025
@kelly-sovacool kelly-sovacool requested a review from kopardev April 10, 2025 21:53
Copy link
Contributor

Choose a reason for hiding this comment

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

@kelly-sovacool .. I did not know about the envsubst command... this is cool!!

@kopardev kopardev merged commit 939dfd5 into main Apr 14, 2025
5 checks passed
@kopardev kopardev deleted the version-spooker branch April 14, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASPEN RepoName

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants