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.

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

feat: Add URL validation contract and error messages #67

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: Add URL validation contract and error messages #67

wants to merge 2 commits into from

Conversation

LyubomirT
Copy link

Description

Proposed Changes

Fixed issue #33 by adding the necessary error messages, classes, and methods to add the UrlValidationContract.

Review Checklist

  • I have read the Contributing.md
  • Tests have been added or updated to reflect the changes (if applicable).
  • An entry has been added to the changelog / My PR does not require a new changelog entry.
  • The documentation in Portuguese has been updated or created, if necessary.
  • If documentation was created, the update of the English file.
  • I have documented my changes in the code, adding docstrings and comments. Instructions
  • The code follows the project's style guidelines and coding standards.
  • All tests pass. Instructions
  • The Pull Request has been tested locally. Instructions
  • There are no merge conflicts.

Additional Comments (optional)

Related Issue

#33

Closes #33

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.

Add UrlValidationContract
1 participant
@LyubomirT