-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore: update hgnc_complete_set download url #100
base: main
Are you sure you want to change the base?
Conversation
WalkthroughThe changes involve updates to the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Snakemake
participant HGNC
User->>Snakemake: Request HGNC data
Snakemake->>HGNC: Download JSON from new URL
HGNC-->>Snakemake: Return JSON data
Snakemake-->>User: Provide processed data
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🔇 Additional comments (2)rules/work/genes/hgnc.smk (1)
The new URL contains a redundant "json/json/" path segment which may be a typo. Please verify if this is the correct path structure. Run the following script to verify the URL accessibility and correct path: ✅ Verification successfulURL path structure is correct and accessible The current URL with "json/json/" in the path is correct and accessible, returning a 200 status code with a valid JSON file (33MB). The alternative URL without the redundant "json" returns a 404 error. The file was last modified on December 6, 2024, indicating it's actively maintained. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check URL accessibility and compare with potential alternatives
# Test the current URL
echo "Testing current URL..."
curl -sI "https://storage.googleapis.com/public-download-files/hgnc/json/json/hgnc_complete_set.json"
# Test without redundant json/
echo -e "\nTesting alternative URL..."
curl -sI "https://storage.googleapis.com/public-download-files/hgnc/json/hgnc_complete_set.json"
Length of output: 1314 download_urls.yml (1)
The new URL entry is well-configured with appropriate flags:
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 (
|
Please check for formatting and linting issues. You can use
|
Summary by CodeRabbit
New Features
Bug Fixes
Chores
skip_upstream_check: true
for reliability during tests.