-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update quatradis to 1.3.1 #51027
Update quatradis to 1.3.1 #51027
Conversation
📝 Walkthrough📝 WalkthroughWalkthroughThe pull request updates the Changes
Possibly related PRs
Thank 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? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
recipes/quatradis/meta.yaml (1)
16-16
: Remove trailing spaces.There are trailing spaces on this line. Please remove them to improve code cleanliness.
Note: The static analysis tool also reported a syntax error for the '%' character on line 1. This is a false positive as the '%' is part of the Jinja2 template syntax commonly used in Conda recipes.
🧰 Tools
🪛 yamllint
[error] 16-16: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- recipes/quatradis/meta.yaml (3 hunks)
🧰 Additional context used
🪛 yamllint
recipes/quatradis/meta.yaml
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
🔇 Additional comments (7)
recipes/quatradis/meta.yaml (7)
1-1
: LGTM: Version update is correct.The package version has been correctly updated to 1.3.1, which aligns with the PR objectives.
🧰 Tools
🪛 yamllint
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
13-15
: LGTM: Build script and pinning improvements.The changes to the build script and pinning are beneficial:
- The more verbose pip installation command (
-vvv --no-deps --no-build-isolation --no-cache-dir
) provides better isolation and debugging capabilities.- Adjusting the max_pin to "x" allows for more flexible version compatibility while still maintaining major version stability.
These changes should improve the build process and package management.
21-22
: Verify compatibility with different Python and Cython versions.The specific version constraints for Python and Cython have been removed. While this provides more flexibility, it's important to ensure that the package remains compatible with a wide range of versions. Consider the following:
- Verify that the package works correctly with the latest Python and Cython versions.
- If there are known incompatibilities with certain versions, consider adding appropriate version constraints.
- Update the package documentation to clearly state the supported Python and Cython versions.
32-33
: LGTM: Dependency updates improve compatibility and reduce package size.The changes to the run requirements are beneficial:
- Specifying dendropy==4.6 ensures compatibility with a known working version.
- Changing seaborn to seaborn-base reduces dependencies and package size.
- Using snakemake-minimal instead of snakemake also reduces dependencies and package size.
These changes should improve the package's compatibility and installation efficiency.
Also applies to: 44-44
57-57
: LGTM: Addition of dev_url improves metadata.The addition of the dev_url pointing to the GitHub repository is a good practice. It improves the package metadata and makes it easier for users to find the source code and contribute to the project.
66-67
: LGTM: Improved formatting of identifiers.The reformatting of the DOI entry under the identifiers section improves the structure of the metadata and aligns with best practices for specifying identifiers in Conda recipes.
9-9
: Verify the SHA256 checksum.The SHA256 checksum has been updated for the new version. Please ensure that this checksum is correct for the 1.3.1 release tarball.
Run the following script to verify the checksum:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
recipes/quatradis/meta.yaml (1)
16-16
: Remove trailing spacesThere are trailing spaces on this line. Please remove them to improve code cleanliness.
Apply this change:
- run_exports: + run_exports:🧰 Tools
🪛 yamllint
[error] 16-16: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- recipes/quatradis/meta.yaml (4 hunks)
🧰 Additional context used
🪛 yamllint
recipes/quatradis/meta.yaml
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
🔇 Additional comments (7)
recipes/quatradis/meta.yaml (7)
1-1
: LGTM: Version updateThe package version has been correctly updated to 1.3.1, which aligns with the PR objectives.
🧰 Tools
🪛 yamllint
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
9-9
: LGTM: Source URL and checksum updateThe SHA256 checksum has been updated to match the new version's source tarball. The source URL is also implicitly updated through the jinja2 template.
21-22
: Verify compatibility with latest Python and Cython versionsThe specific version constraints for Python and Cython have been removed from the host requirements. While this allows for more flexibility, it's important to ensure compatibility with the latest versions.
Please verify that the package builds and functions correctly with the latest Python and Cython versions. Consider adding appropriate version constraints if any compatibility issues are found.
Line range hint
27-45
: Verify impact of dependency changesSeveral changes have been made to the run requirements:
- The specific version constraint for Cython has been removed.
- dendropy version is now pinned to 4.6.
- seaborn has been changed to seaborn-base.
- snakemake has been changed to snakemake-minimal.
These changes might affect the package's functionality and compatibility.
Please verify that:
- The package works correctly with the latest Cython version.
- The pinned version of dendropy (4.6) is appropriate and doesn't introduce any issues.
- seaborn-base provides all the necessary functionality required by quatradis.
- snakemake-minimal includes all the features needed by quatradis.
Consider running the package's test suite with these updated dependencies to ensure everything functions as expected.
🧰 Tools
🪛 yamllint
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
58-58
: LGTM: Addition of dev_urlThe dev_url has been added, pointing to the GitHub repository. This is a positive change that improves the package metadata by providing a direct link to the development repository.
69-70
: LGTM: DOI entry updateThe DOI entry has been correctly reformatted under the 'identifiers' section. This change improves the structure of the metadata without altering the DOI itself.
13-15
: Verify the impact of the pinning changeThe build script has been updated to use a more verbose pip installation command, which is good for debugging purposes.
However, the maximum pinning for the quatradis subpackage has been changed from "x.x" to "x". This change might affect compatibility with other packages.
Please verify that this pinning change doesn't introduce any compatibility issues with dependent packages.
Update
quatradis
: 1.3.0 → 1.3.1recipes/quatradis
(click to view/edit other files)@quadram-institute-bioscience
This pull request was automatically generated (see docs).