Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

fix: email TLD can be longer than 3 chars #928

Merged
merged 2 commits into from
Feb 21, 2022
Merged

fix: email TLD can be longer than 3 chars #928

merged 2 commits into from
Feb 21, 2022

Conversation

SanketD92
Copy link
Contributor

Issue #, if available:
#926

Description of changes:
Increasing TLD limit to allow more valid cases.

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully tested with your changes locally?

AS review ticket id:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@SanketD92 SanketD92 requested a review from a team as a code owner February 18, 2022 19:16
@codecov
Copy link

codecov bot commented Feb 18, 2022

Codecov Report

Merging #928 (7702bb6) into develop (1465431) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #928   +/-   ##
========================================
  Coverage    51.43%   51.43%           
========================================
  Files          295      295           
  Lines        16787    16787           
  Branches      2596     2596           
========================================
  Hits          8634     8634           
  Misses        7166     7166           
  Partials       987      987           

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 1465431...7702bb6. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants