Skip to content

Code Cleanup #221

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 18, 2025
Merged

Code Cleanup #221

merged 1 commit into from
Oct 18, 2025

Conversation

@mgomes
Copy link
Owner

@mgomes mgomes commented Oct 18, 2025

  • A lot of drivers were having to validate URI
  • We had an implicit runtime requirement on ActiveSupport as we used present? in a couple spots

@mgomes mgomes merged commit 64da504 into master Oct 18, 2025
7 checks passed
@mgomes mgomes deleted the mgomes/refactor branch October 18, 2025 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mgomes