Skip to content

fix: remove paramiko dependency as issue has been fixed#3110

Merged
johanneskoester merged 4 commits intosnakemake:mainfrom
fgvieira:paramiko_ver
Oct 13, 2024
Merged

fix: remove paramiko dependency as issue has been fixed#3110
johanneskoester merged 4 commits intosnakemake:mainfrom
fgvieira:paramiko_ver

Conversation

@fgvieira
Copy link
Copy Markdown
Contributor

@fgvieira fgvieira commented Oct 3, 2024

QC

  • The PR contains a test case for the changes or the changes are already covered by an existing test case.
  • The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Summary by CodeRabbit

  • Chores
    • Cleaned up dependencies in the test environment configuration.
    • Commented out the paramiko dependency as a temporary workaround.
    • Corrected indentation for the nodejs dependency.
    • Updated pip section to include new plugins for snakemake with specific version requirements.
    • Added a comment for future removal of the version constraint on the peppy dependency.
    • Removed Unix-only dependencies to enhance compatibility with Windows environments.
  • New Features
    • Added a new job for building a Docker container image in the workflow.
  • Bug Fixes
    • Corrected the step name from "Setup snakemke environment" to "Setup snakemake environment" for clarity.

Loading
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