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

Add --no-config option #122

Merged
merged 1 commit into from
Jul 23, 2016
Merged

Add --no-config option #122

merged 1 commit into from
Jul 23, 2016

Conversation

msabramo
Copy link
Contributor

@msabramo msabramo commented Jul 23, 2016

This lets us get rid of YAML configuration file couldn't be found at /no/config. Using defaults. warnings; especially when running the unit tests.

$ bundle exec rake test
/usr/local/Cellar/ruby/2.3.1/bin/ruby -I"lib:lib:test" -I"/Users/marca/dev/git-repos/aptly_cli/.bundle/gems/gems/rake-10.5.0/lib" "/Users/marca/dev/git-repos/aptly_cli/.bundle/gems/gems/rake-10.5.0/lib/rake/rake_test_loader.rb" "test/test_aptly_cli_configure.rb" "test/test_aptly_command.rb" "test/test_aptly_file.rb" "test/test_aptly_misc.rb" "test/test_aptly_package.rb" "test/test_aptly_publish.rb" "test/test_aptly_repo.rb" "test/test_aptly_snapshot.rb"
/Users/marca/dev/git-repos/aptly_cli/test/minitest_helper.rb:4:in `<top (required)>': [DEPRECATION] ::[] is deprecated. Use ::new instead.
Run options: --seed 26736

.................................................................

Finished in 0.460966s, 141.0082 runs/s, 232.1212 assertions/s.

65 runs, 107 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for Unit Tests to /Users/marca/dev/git-repos/aptly_cli/coverage. 636 / 658 LOC (96.66%) covered.
[Coveralls] Outside the CI environment, not sending data.

This lets us get rid of `YAML configuration file couldn't be found at
/no/config. Using defaults.` warnings; especially when running the unit
tests.

```
$ bundle exec rake test
/usr/local/Cellar/ruby/2.3.1/bin/ruby -I"lib:lib:test" -I"/Users/marca/dev/git-repos/aptly_cli/.bundle/gems/gems/rake-10.5.0/lib" "/Users/marca/dev/git-repos/aptly_cli/.bundle/gems/gems/rake-10.5.0/lib/rake/rake_test_loader.rb" "test/test_aptly_cli_configure.rb" "test/test_aptly_command.rb" "test/test_aptly_file.rb" "test/test_aptly_misc.rb" "test/test_aptly_package.rb" "test/test_aptly_publish.rb" "test/test_aptly_repo.rb" "test/test_aptly_snapshot.rb"
/Users/marca/dev/git-repos/aptly_cli/test/minitest_helper.rb:4:in `<top (required)>': [DEPRECATION] ::[] is deprecated. Use ::new instead.
Run options: --seed 26736

.................................................................

Finished in 0.460966s, 141.0082 runs/s, 232.1212 assertions/s.

65 runs, 107 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for Unit Tests to /Users/marca/dev/git-repos/aptly_cli/coverage. 636 / 658 LOC (96.66%) covered.
[Coveralls] Outside the CI environment, not sending data.
```
@coveralls
Copy link

coveralls commented Jul 23, 2016

Coverage Status

Coverage increased (+0.008%) to 97.222% when pulling 5b93f20 on msabramo:no-config into e0b3638 on sepulworld:master.

@sepulworld sepulworld merged commit 5b93f20 into sepulworld:master Jul 23, 2016
@sepulworld
Copy link
Owner

Thank you! Much cleaner output 💯

@msabramo msabramo deleted the no-config branch July 23, 2016 08:04
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.

3 participants