Skip to content

chore: remove unused REMITLY_CORRIDORS_URL constant from scrapers/remitly.py#18

Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1772896016-remove-unused-remitly-corridors-url
Open

chore: remove unused REMITLY_CORRIDORS_URL constant from scrapers/remitly.py#18
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1772896016-remove-unused-remitly-corridors-url

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

chore: remove unused REMITLY_CORRIDORS_URL constant

Summary

Removes the REMITLY_CORRIDORS_URL module-level constant from scrapers/remitly.py (line 21). This URL was speculatively added during initial code generation but is never referenced by any function, method, or import in the codebase. The sibling constant REMITLY_ESTIMATE_URL remains as it is actively used by _get_estimate().

Review & Testing Checklist for Human

  • Confirm REMITLY_CORRIDORS_URL is not accessed dynamically anywhere (e.g., via globals(), getattr, or string-based lookups) — a repo-wide grep for CORRIDORS should return zero hits
  • Verify no external consumers (scripts, notebooks, other services) import this constant from scrapers.remitly

Notes

…itly.py

Co-Authored-By: bot_apk <apk@cognition.ai>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants