Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(renovate): Allow channels in environment.ymls #5400

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

edmundmiller
Copy link
Contributor

@edmundmiller edmundmiller commented Mar 26, 2024

No clue if this will work, it passed locally 馃し馃徎

Adds a regex to fix not finding the dependency if :: and a channel is in the conda dependency.

@edmundmiller edmundmiller self-assigned this Mar 26, 2024
@edmundmiller edmundmiller marked this pull request as ready for review March 26, 2024 17:47
@edmundmiller
Copy link
Contributor Author

The scope has changed several times is why, I first set out to add the comments to the rnaseq modules, then I realized we had added back in all the channels (which I think helps conda figure the environment out quicker and more consistently), so I fixed the regex, then I was debating adding everything with an automated script.

Maybe that just turns into an nf-core modules renovate command, and it adds renovate comments to the module you want 馃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants