Update r-oncopharmadb to 1.8.1 #52029
Conversation
📝 WalkthroughWalkthroughThe pull request introduces updates to the metadata file for the 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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
recipes/r-oncopharmadb/meta.yaml (1)
Package name mismatch needs to be corrected
There are inconsistencies in the package naming that need to be addressed:
- The conda package name is set as
r-oncopharmadb- The GitHub repository has two different names in the same file:
oncoPharmaDB(ingithubvariable andhomeURL)pharmOncoX(in source URL and test command)- The R package being tested is
pharmOncoXThe source URL and test command are correct as they use
pharmOncoX, which is the actual R package name. The conda package name and GitHub repository references should be updated to match:
- Change package name from
r-oncopharmadbtor-pharmonco- Update GitHub references to consistently use
pharmOncoX🔗 Analysis chain
Line range hint
43-43: Verify package name consistency.There's a potential inconsistency in naming:
- Package name:
r-oncopharmadb- GitHub repo:
oncoPharmaDB- Test command:
pharmOncoX- Source URL:
pharmOncoXPlease verify that
pharmOncoXis the correct R package name for testing.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check repository contents for R package name # Expected results: Find the actual R package name in DESCRIPTION file echo "Checking R package name in repository..." curl -sL "https://raw.githubusercontent.com/sigven/pharmOncoX/1.8.1/DESCRIPTION" | grep "Package:"Length of output: 302
Script:
#!/bin/bash # Let's verify the package name and repository details from meta.yaml cat recipes/r-oncopharmadb/meta.yaml # Also check if there are any other references to the package name in the recipe rg -i "pharmonco|oncopharma" recipes/r-oncopharmadb/Length of output: 1598
🧰 Tools
🪛 yamllint
[error] 1-1: syntax error: found character '%' that cannot start any token
(syntax)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
recipes/r-oncopharmadb/meta.yaml(2 hunks)
🔇 Additional comments (2)
recipes/r-oncopharmadb/meta.yaml (2)
2-2: LGTM: Version update is consistent with PR objectives.
The version bump to 1.8.1 matches the intended update described in the PR.
10-11: Verify source integrity and accessibility.
Let's verify the source URL is accessible and the SHA256 checksum matches the tarball.
Update
r-oncopharmadb: 1.7.0 → 1.8.1recipes/r-oncopharmadb(click to view/edit other files)@sigvenThis pull request was automatically generated (see docs).