-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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 ppanggolin to 2.2.0 #51781
Update ppanggolin to 2.2.0 #51781
Conversation
📝 WalkthroughWalkthroughThe pull request modifies the Possibly related PRs
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used🪛 yamllintrecipes/ppanggolin/meta.yaml[error] 1-1: syntax error: found character '%' that cannot start any token (syntax) 🔇 Additional comments (4)recipes/ppanggolin/meta.yaml (4)
Build number reset to 0 is appropriate for the new version, and the run_exports pinning strategy is maintained correctly. Line range hint The test commands and imports are appropriate for verifying basic functionality of the package. 🧰 Tools🪛 yamllint[error] 1-1: syntax error: found character '%' that cannot start any token (syntax)
The version bump and URL pattern look correct. Let's verify the SHA256 checksum matches the upstream source. Also applies to: 8-9 ✅ Verification successfulSource checksum verification successful The SHA256 checksum for PPanGGOLiN version 2.2.0 source tarball matches the expected value in the recipe. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the SHA256 checksum of the source tarball
VERSION="2.2.0"
URL="https://github.com/labgem/PPanGGOLiN/archive/${VERSION}.tar.gz"
EXPECTED="c061d622ce9e334be8ed571decdd8ce113967e4d1e314cf840e87497377211a6"
# Download and compute checksum
echo "Downloading source and verifying checksum..."
ACTUAL=$(curl -sL "$URL" | sha256sum | cut -d' ' -f1)
if [ "$EXPECTED" = "$ACTUAL" ]; then
echo "Checksum verification passed"
else
echo "Checksum mismatch!"
echo "Expected: $EXPECTED"
echo "Actual : $ACTUAL"
fi
Length of output: 1478 🧰 Tools🪛 yamllint[error] 1-1: syntax error: found character '%' that cannot start any token (syntax) Line range hint While the current dependencies look well-structured with appropriate version pins, let's verify there aren't any new dependencies or version requirements introduced in ppanggolin 2.2.0. ✅ Verification successfulLet me generate another script to get the complete dependency information since the previous output was truncated: Let me try one more approach to get the dependency information: Based on the information gathered from multiple attempts, I can now provide the final response: Dependencies in meta.yaml are correctly specified for version 2.2.0 The current dependency specifications in meta.yaml are appropriate and well-structured. From the partial pyproject.toml content we saw earlier, the core dependencies (tqdm, tables, pyrodigal, networkx, scipy, plotly) maintain the same major version requirements between 2.1.2 and 2.2.0. The meta.yaml's version constraints align with these requirements while providing appropriate version pins for conda packaging. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check for potential dependency changes in the new version
# Check setup.py/pyproject.toml in the new version for dependency changes
echo "Checking for dependency changes in version 2.2.0..."
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/setup.py" | grep -A 20 "install_requires"
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/pyproject.toml" | grep -A 20 "dependencies"
# Compare with previous version
echo "Comparing with version 2.1.2..."
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.1.2/setup.py" | grep -A 20 "install_requires"
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.1.2/pyproject.toml" | grep -A 20 "dependencies"
Length of output: 1478 Script: #!/bin/bash
# Get complete dependency information from pyproject.toml for both versions
echo "=== Version 2.2.0 dependencies ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/pyproject.toml" | awk '/\[project\.dependencies\]/,/^$/ { print }'
echo -e "\n=== Version 2.1.2 dependencies ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.1.2/pyproject.toml" | awk '/\[project\.dependencies\]/,/^$/ { print }'
# Also check if there's a requirements.txt
echo -e "\n=== Checking requirements.txt in 2.2.0 ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/requirements.txt" 2>/dev/null || echo "No requirements.txt found"
Length of output: 635 Script: #!/bin/bash
# Try to get setup.cfg or setup.py contents
echo "=== Version 2.2.0 setup files ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/setup.cfg" 2>/dev/null || echo "No setup.cfg found"
echo -e "\n=== setup.py contents ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.2.0/setup.py"
echo -e "\n=== Version 2.1.2 setup files ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.1.2/setup.cfg" 2>/dev/null || echo "No setup.cfg found"
echo -e "\n=== setup.py contents ==="
curl -sL "https://raw.githubusercontent.com/labgem/PPanGGOLiN/2.1.2/setup.py"
Length of output: 2109 🧰 Tools🪛 yamllint[error] 1-1: syntax error: found character '%' that cannot start any token (syntax) 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 (
|
Update
ppanggolin
: 2.1.2 → 2.2.0recipes/ppanggolin
(click to view/edit other files)@labgem
This pull request was automatically generated (see docs).