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

Clean up Travis logs #2357

Merged
merged 4 commits into from
Jan 23, 2018
Merged

Clean up Travis logs #2357

merged 4 commits into from
Jan 23, 2018

Conversation

voodoorai2000
Copy link
Member

@voodoorai2000 voodoorai2000 commented Jan 22, 2018

What

  • Remove asset precompilation output from travis logs
  • Reduce travis' knapsack log level to info
  • Fix RSpec's should deprecation warning
  • Remove duplicate dev_seed execution

Test

  • Removed duplicate execution of dev seeds in specs

Deployment

  • As usual

Warnings

  • None

Asset precompilation generates a lot of logs, making it slower to
scroll down to see the commonly relevant information, knowing what spec
has failed

Redirecting standard error and standard output to dev/null helps reach
this information faster
Using `stub` from rspec-mocks' old `:should` syntax without explicitly
enabling the syntax is deprecated. Use the new `:expect` syntax or
explicitly enable `:should` instead
@voodoorai2000 voodoorai2000 changed the title [WIP] Clean up Travis logs Clean up Travis logs Jan 23, 2018
@voodoorai2000 voodoorai2000 merged commit c96e8d1 into master Jan 23, 2018
@voodoorai2000 voodoorai2000 deleted the travis-logs branch January 23, 2018 20:31
clairezed pushed a commit to CDJ11/CDJ that referenced this pull request Jun 26, 2018
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