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

Let test_datarange show only a warning instead of fail on AssertionError #122

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Let test_datarange show only a warning instead of fail on AssertionError #122

merged 1 commit into from
Sep 11, 2019

Conversation

frenzymadness
Copy link
Contributor

I think that the subject says it all.

Fixes: #121

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Feb 14, 2019
@misterdorm misterdorm added status: ready for deploy code ready to be released in next deploy and removed status: code review request requesting a community code review or review from Twilio labels Feb 25, 2019
@thinkingserious thinkingserious merged commit 397b3a6 into sendgrid:master Sep 11, 2019
@thinkingserious
Copy link
Contributor

Hello @frenzymadness,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@misterdorm misterdorm misterdorm approved these changes

Assignees
No one assigned
Labels
status: ready for deploy code ready to be released in next deploy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Test of date range in license file
3 participants
@frenzymadness @thinkingserious @misterdorm