Skip to content
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

Add a new constraint to validate if a string is a website #30

Merged

Conversation

robertoamadeuneto
Copy link

Proposed Changes

  • This change implements a new Constraint called Website that validates if a given String is a valid website.

@codecov-io
Copy link

Codecov Report

Merging #30 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #30      +/-   ##
============================================
+ Coverage     90.19%   90.23%   +0.03%     
  Complexity      120      120              
============================================
  Files            73       73              
  Lines          1030     1034       +4     
  Branches        437      438       +1     
============================================
+ Hits            929      933       +4     
  Misses           18       18              
  Partials         83       83
Impacted Files Coverage Δ Complexity Δ
...n/kotlin/org/valiktor/functions/StringFunctions.kt 100% <100%> (ø) 0 <0> (ø) ⬇️
...kotlin/org/valiktor/constraints/TextConstraints.kt 100% <100%> (ø) 0 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 622a2b9...2932bd5. Read the comment docs.

@rodolphocouto rodolphocouto merged commit f615863 into valiktor:master Feb 8, 2020
@rodolphocouto rodolphocouto added this to the 0.10.0 milestone Feb 8, 2020
@rodolphocouto rodolphocouto added the feature New feature or request label Mar 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Development

Successfully merging this pull request may close these issues.

None yet

3 participants