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

Loosen version requirement email_validator #1716

Merged
merged 1 commit into from Jul 20, 2023
Merged

Conversation

JPBergsma
Copy link
Contributor

In setup.py, we state that only version 1.* of email_validator is acceptable. Yet in requirements.txt we state that the required version is 2.0.0.post2 Therefore, I have made the version requirement in setup.py more flexible.

In setup.py, we state that only version 1.* of email_validator is acceptable.
Yet in requirements.txt we state that the required version is 2.0.0.post2
Therefore, I have made the version requirement in setup.py more flexible.
@codecov
Copy link

codecov bot commented Jul 20, 2023

Codecov Report

Merging #1716 (e8327a0) into master (6a30267) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1716      +/-   ##
==========================================
- Coverage   90.92%   90.90%   -0.03%     
==========================================
  Files          74       74              
  Lines        4608     4608              
==========================================
- Hits         4190     4189       -1     
- Misses        418      419       +1     
Flag Coverage Δ
project 90.90% <ø> (-0.03%) ⬇️
validator 90.79% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

@JPBergsma JPBergsma marked this pull request as ready for review July 20, 2023 12:10
@JPBergsma JPBergsma merged commit a8e28e1 into master Jul 20, 2023
11 checks passed
@JPBergsma JPBergsma deleted the JPBergsma-patch-2 branch July 20, 2023 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant