coderabbitai[bot]
commented
2 days ago 📝 Walkthrough
## Walkthrough The pull request updates the `meta.yaml` file for the Snakemake package. The version number has been changed from `8.24.1` to `8.25.0`, necessitating an update to the SHA256 checksum to reflect the new version's source tarball. The `source` section has been revised to point to the updated URL that corresponds to the new version. Additionally, the `requirements` section has been adjusted to ensure compatibility with the `snakemake-minimal` dependency for the new version. The `test` commands remain unchanged, continuing to verify the installed version of Snakemake, while the `about` section retains its original content, providing information about the package. ## Possibly related PRs - #51586: This PR updates the version of Snakemake from `8.24.0` to `8.24.1`, which is directly related to the main PR that updates the version from `8.24.1` to `8.25.0`. ## Suggested reviewers - johanneskoester📜 Recent review details
**Configuration used: CodeRabbit UI** **Review profile: CHILL**📥 Commits
Files that changed from the base of the PR and between 0156829d2bf2d03eee2b99d46e34bf34d0eb7f7f and 0b58c62cfb9643ded9d48502a5ffceb1009227fd.📒 Files selected for processing (1)
* recipes/snakemake/meta.yaml (1 hunks)🔇 Additional comments (5)
recipes/snakemake/meta.yaml (5)
Line range hint `35-41`: **Test commands look comprehensive** The test suite includes: - Version verification - Report generation to validate graphviz dependencies - Proper cleanup of test artifacts Also applies to: 127-131🧰 Tools
🪛 yamllint
[error] 3-3: syntax error: found character '%' that cannot start any token (syntax)🧰 Tools
🪛 yamllint
[error] 3-3: syntax error: found character '%' that cannot start any token (syntax)🧰 Tools
🪛 yamllint
[error] 3-3: syntax error: found character '%' that cannot start any token (syntax)✅ Verification successful
**Dependencies are correctly aligned with setup.cfg** After comparing the dependencies between setup.cfg and meta.yaml: - All core dependencies from setup.cfg are present in meta.yaml with matching version constraints - The only differences are additional dependencies in meta.yaml (pandas, slack_sdk, eido, peppy, pygments) which are expected as they are specific to the full Snakemake package rather than the minimal installation - Python version constraints and build dependencies (pip, setuptools) are appropriately specified in meta.yaml🏁 Scripts executed
The following scripts were executed for the analysis: Script: ```shell #!/bin/bash # Fetch and compare dependencies from setup.cfg curl -sL "https://raw.githubusercontent.com/snakemake/snakemake/v8.25.0/setup.cfg" | awk '/install_requires =/{flag=1;next}/^[[:space:]]*$/{flag=0}flag' ``` Length of output: 877 --- Script: ```shell #!/bin/bash # Get dependencies from meta.yaml for comparison cat recipes/snakemake/meta.yaml | awk '/requirements:/,/test:/' | grep -E '^\s*-\s*[^#]*' | sed 's/^\s*-\s*//' ``` Length of output: 829Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
aliciaaevans
BiocondaBot
Update
snakemake: 8.24.1 → 8.25.0recipes/snakemake(click to view/edit other files)This pull request was automatically generated (see docs).