Skip to content

Conversation

@k1bs
Copy link
Collaborator

@k1bs k1bs commented Feb 4, 2026

Whoopsie.

@k1bs k1bs requested a review from fastjames February 4, 2026 19:02
@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

Summary by CodeRabbit

  • Chores
    • Updated the default API endpoint from https://api.datanexus.com to https://datanexus.darthealth.com. This changes where the client connects by default when no custom base URL is specified. Documentation and configuration examples have been updated accordingly.

Walkthrough

Updated the default base URL for the DataNexus Ruby client from https://api.datanexus.com to https://datanexus.darthealth.com across documentation, configuration examples, and the configuration constant default value.

Changes

Cohort / File(s) Summary
Default Base URL Updates
README.md, lib/data_nexus.rb, lib/data_nexus/configuration.rb
Changed the default API endpoint URL from https://api.datanexus.com to https://datanexus.darthealth.com in README documentation, configuration example comments, and the DEFAULT_BASE_URL constant in the Configuration class.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description 'Whoopsie.' is extremely vague and provides no meaningful information about what was changed, why, or how the fix addresses the issue. Expand the description to explain the change—what URL was updated, where, and why this fix is necessary for the DataNexus Ruby client.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix default URL' accurately describes the main change—updating the default base URL from api.datanexus.com to datanexus.darthealth.com across the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch k1bs/fix-default-url

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@fastjames fastjames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍰

@k1bs k1bs merged commit c5cace0 into main Feb 4, 2026
7 checks passed
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.

2 participants