-
Notifications
You must be signed in to change notification settings - Fork 55
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
Refactors and addressed some issues. #19
Conversation
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
No longer raises error when supplied a domain without a valid public suffix. Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Less ways for domainatrix to raise errors. Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Signed-off-by: Patrick Ma <[email protected]>
Solved #16 and #13 Signed-off-by: Patrick Ma <[email protected]>
hey guys, I'm finally getting around to pulling in requests and doing a new release of domainatrix. Can you take another look at this and update the pull request? Thanks! |
@PatrickMA - Actually I have been using your fork and there are some issues with Domain parsing: Can you please look into the same? Issue does not happen with Pauldix branch.. |
Before integrating this refactors, please take a look at this issue: Thanks! |
Split up a large method into smaller methods.
Simplified conditions.
Removed white space.
Domains with no public suffixes now return a empty string as the public_suffix value instead of raising errors. (Requested by issue #16)
Added a valid? method to the URL class. (Requested by issue #14)
Tested against issue #15.
Rescued against Addressable::URI::InvalidURIError exception.
Defaults to http scheme. #13.
Includes a couple other refactors, see diff.
99.25% test coverage. Took simplecov warnings from 16 to 10.
Rebased on Jamie Cobbett's 0.0.10 version of the gem.
Bumped version to 0.0.11 after covering issue #16.
Bumped version to 0.0.12 after covering issue #14.
Cheers.