Skip to content

chore: remove unused XE_GRAPH_URL constant from scrapers/xe.py#14

Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1772849792-remove-unused-xe-graph-url
Open

chore: remove unused XE_GRAPH_URL constant from scrapers/xe.py#14
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1772849792-remove-unused-xe-graph-url

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

chore: remove unused XE_GRAPH_URL constant from scrapers/xe.py

Summary

Removes the unused XE_GRAPH_URL module-level constant from scrapers/xe.py. This midmarket converter API URL was speculatively defined during initial code generation but is never referenced in any scraper method. The sibling constants XE_RATE_URL and XE_SEND_MONEY_URL remain, as they are actively used by _get_rate_from_page() and _get_send_money_quote() respectively.

Review & Testing Checklist for Human

  • Verify via repo-wide grep that XE_GRAPH_URL is not referenced anywhere (including dynamic/string-based lookups or external consumers importing it)

Notes

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