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

dont show config when asked #58

Merged
merged 1 commit into from
Dec 5, 2016
Merged

dont show config when asked #58

merged 1 commit into from
Dec 5, 2016

Conversation

davidx
Copy link
Contributor

@davidx davidx commented Dec 5, 2016

This omits displaying the initial thank you and config settings on request.

show_config: false # default is true

example: davidx/prtester#330

@thumbot
Copy link
Collaborator

thumbot commented Dec 5, 2016

Thanks @davidx!

Settings
---
minimum_reviewers: 1
build_steps:
- bundle install
- bundle exec ruby test/test.rb
merge: true
org_mode: true
timeout: 1805
shell: "/bin/bash"
env:
  BUNDLE_GEMFILE: Gemfile

@thumbot
Copy link
Collaborator

thumbot commented Dec 5, 2016

show_config f59f2c9 ➡️ master ab5e605 ✅ completed
Looks good! 👍
✅ MERGE

Started at: 2016-12-05 15:47
Duration: 1 seconds.
Result: OK
Message: Merge Success: show_config f59f2c9 onto target branch: master ab5e605
Exit Code: OK

📄







  Merge made by the 'recursive' strategy.
 app.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)




✅ BUNDLE_INSTALL

Started at: 2016-12-05 15:47
Duration: 0 seconds.
Result: OK
Message: OK
Exit Code: 0

📄


cd /tmp/thumbs/master.ab5e605.show_config.f59f2c9; bundle install




  Using rake 11.3.0
Using concurrent-ruby 1.0.2
Using i18n 0.7.0
Using minitest 5.10.1
Using thread_safe 0.3.5
Using builder 3.2.2
Using erubis 2.7.0
Using mini_portile2 2.1.0
Using rack 2.0.1
Using public_suffix 2.0.4
Using mime-types-data 3.2016.0521
Using safe_yaml 1.0.4
Using diff-lcs 1.2.5
Using unf_ext 0.0.7.2
Using dust 0.1.6
Using excon 0.54.0
Using multipart-post 2.0.0
Using git 1.3.0
Using graphql 1.1.0
Using hashdiff 0.3.1
Using hijack 0.2.1
Using http-form_data 1.0.1
Using http_parser.rb 0.6.0
Using kgio 2.10.0
Using log4r 1.1.10
Using method_source 0.8.2
Using mustermann 1.0.0.beta2
Using netrc 0.11.0
Using power_assert 0.4.1
Using thor 0.19.4
Using raindrops 0.17.0
Using rspec-support 3.5.0
Using tilt 2.0.5
Using vcr 3.0.3
Using bundler 1.13.6
Using tzinfo 1.2.2
Using nokogiri 1.6.8.1
Using rack-test 0.6.3
Using rack-protection 2.0.0.beta2
Using addressable 2.5.0
Using mime-types 3.1
Using crack 0.4.3
Using unf 0.1.4
Using faraday 0.10.0
Using test-unit 3.2.3
Using unicorn 5.2.0
Using rspec-core 3.5.4
Using rspec-expectations 3.5.0
Using rspec-mocks 3.5.0
Using activesupport 5.0.0.1
Using loofah 2.0.3
Using xpath 2.0.0
Using sinatra 2.0.0.beta2
Using webmock 2.3.0
Using domain_name 0.5.20161129
Using sawyer 0.8.1
Using rails-dom-testing 2.0.1
Using graphql-client 0.2.3
Using rails-html-sanitizer 1.0.3
Using capybara 2.11.0
Using http-cookie 1.0.3
Using octokit 4.6.2
Using actionview 5.0.0.1
Using http 2.1.0
Using actionpack 5.0.0.1
Using railties 5.0.0.1
Using rspec-rails 3.5.2
Bundle complete! 22 Gemfile dependencies, 67 gems now installed.
Use `bundle show [gemname]` to see where a bundled gem is installed.





✅ BUNDLE_EXEC_RUBY_TEST/TEST.RB

Started at: 2016-12-05 15:47
Duration: 226 seconds.
Result: OK
Message: OK
Exit Code: 0

📄


cd /tmp/thumbs/master.ab5e605.show_config.f59f2c9; bundle exec ruby test/test.rb




  /tmp/thumbs/master.ab5e605.show_config.f59f2c9/test/test_common.rb:127: warning: key :record is duplicated and overwritten on line 127
Loaded suite test/test
Started
...........................................................................

Finished in 218.898532547 seconds.
------
75 tests, 203 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
------
0.34 tests/s, 0.93 assertions/s





⬜ 0 of 1 Code reviews from organization basho-labs

@paegun
Copy link

paegun commented Dec 5, 2016

+1

Since the intro_text is cheap to make, I'm not complaining that it needn't be made :)

@thumbot
Copy link
Collaborator

thumbot commented Dec 5, 2016

✅ 1 of 1 Code reviews from organization basho-labs

Since the intro_text is cheap to make, I'm not complaining that it needn't be made :)

@thumbot
Copy link
Collaborator

thumbot commented Dec 5, 2016

Merging and closing this pr

@thumbot thumbot merged commit d84263f into master Dec 5, 2016
@thumbot
Copy link
Collaborator

thumbot commented Dec 5, 2016

Successfully merged basho-labs/thumbs/pulls/58 (f59f2c9 on to master)

---
:sha: d84263f4e71e21c7ef3c4b0f6d913f8c20cf051c
:merged: true
:message: Pull Request successfully merged

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