Skip to content

handle prek not being exposed to $PATH#4238

Merged
mashehu merged 5 commits intonf-core:devfrom
mashehu:fix-prek-missing-in-path
Apr 28, 2026
Merged

handle prek not being exposed to $PATH#4238
mashehu merged 5 commits intonf-core:devfrom
mashehu:fix-prek-missing-in-path

Conversation

@mashehu
Copy link
Copy Markdown
Contributor

@mashehu mashehu commented Apr 28, 2026

@mashehu mashehu force-pushed the fix-prek-missing-in-path branch from b47dffe to b27bd38 Compare April 28, 2026 13:25
@mashehu
Copy link
Copy Markdown
Contributor Author

mashehu commented Apr 28, 2026

@nf-core-bot changelog

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

⚠️ Cannot proceed with deployment

  • reviewDecision: REVIEW_REQUIRED

All deployments from forks must have the required reviews before they can proceed. Please ensure this PR has been reviewed and approved before trying again.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 78.38%. Comparing base (33d9944) to head (2dc92aa).
⚠️ Report is 4 commits behind head on dev.

Files with missing lines Patch % Lines
nf_core/pipelines/lint_utils.py 85.71% 1 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mashehu mashehu merged commit 1d3bdda into nf-core:dev Apr 28, 2026
118 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maxulysse maxulysse maxulysse approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mashehu @maxulysse @nf-core-bot