Skip to content

Further reduce tests verbosity #304

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
Merged
merged 3 commits into from
Jun 13, 2018
Merged

Further reduce tests verbosity #304

merged 3 commits into from
Jun 13, 2018

Conversation

btecu
Copy link
Contributor

@btecu btecu commented Jun 13, 2018

Completes #301

@btecu btecu changed the title [WIP] Further reduce tests verbosity Further reduce tests verbosity Jun 13, 2018
@btecu
Copy link
Contributor Author

btecu commented Jun 13, 2018

@jaredcnance this is ready for review.
Reduces log lines (Travis) from 30,870 to 3,201.

There are a lot of warnings and errors that can be cleaned up too.

@jaredcnance jaredcnance merged commit e79b88e into json-api-dotnet:master Jun 13, 2018
jaredcnance added a commit that referenced this pull request Aug 12, 2018
@jaredcnance
Merge pull request #304 from btecu/verbosity
3f4e76f 
Further reduce tests verbosity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@jaredcnance jaredcnance jaredcnance approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@btecu @jaredcnance