aboutsummaryrefslogtreecommitdiffstats
path: root/guides/bug_report_templates
Commit message (Collapse)AuthorAgeFilesLines
* Update bug report templatesRyuta Kamizono2019-04-255-8/+8
| | | | Bump gem version to 6.0.0.rc1.
* Specify sqlite gem version explicitly in version-specific bug report templatesSergey Ponomarev2019-02-042-2/+2
|
* Specify `hosts` in bug report template.yuuji.yaginuma2018-12-251-0/+1
| | | | | | | | | | | | | | | | | | | Host authentication introduced by #33145 allows only "0.0.0.0/0", "::/0" and "localhost" in the development environment. https://github.com/rails/rails/blob/4298df00ae6219b9b5b7c40f281d4fa4d66f4383/railties/lib/rails/application/configuration.rb#L33 But `rack-test` use `example.org` for default host. https://github.com/rack-test/rack-test/blob/6c07bf53adf4a4082e206561f5b9b43142fdc821/lib/rack/test.rb#L13 Therefore, if `hosts` is not specified, host authentication rejects the request. In Travis CI, this problem does not occur because `test` is specified by default in `RAILS_ENV` and `RACK_ENV`. https://docs.travis-ci.com/user/environment-variables/#default-environment-variables If user actually use it, env may not always be specified. Explicitly specify hosts in the file so that it works in any environment.
* Revert "Merge pull request #34421 from ↵Ryuta Kamizono2018-11-112-2/+2
| | | | | | | albertoalmagro/use-assert-not-instead-of-refute" This reverts commit ab5d5c95ec51c63d58d7e75694af748f165e2228, reversing changes made to ef4f5ef69b5a50c897bbb17f6b36b729dff12e6c.
* Prefer assert_not instead of refute [ci skip]Alberto Almagro2018-11-102-2/+2
| | | | | | | | As mentioned in the contributing to Ruby on Rails guides, we should prefer `assert_not` instead of `refute`. And the best place to start giving example are our bug report templates.. See: https://github.com/rails/rails/blame/master/guides/source/contributing_to_ruby_on_rails.md#L256
* Update bug report templatesRobin Dupret2018-10-2811-87/+11
| | | | | | | | | | Given that Bundler 1.10 was released back in 2015, we can assume that people are at least on this version or a higher one so there is no need to ask people to upgrade. Also, given that Rails 5.0 supports Ruby 2.2+ and given that this version come with Minitest 5.4.3 bundled, we don't need to ensure backward compatibility with Minitest 4.
* use "minitest" consistentlyXavier Noria2018-09-117-7/+7
| | | | | | | | | The name of the minitest library is spelled that way: regular font, and lowercase. Lowercase is used even at the beginning of sentences, see http://docs.seattlerb.org/minitest/ I double-checked this with @zenspider too (thanks!).
* Update bug report templatesbogdanvlviv2018-04-105-5/+5
| | | | Set Rails 5.2.0 in templates
* Update bug report templatesbogdanvlviv2018-02-176-7/+8
| | | | | | Prepare bug report templates for Rails 6.0 development Add missing `require "active_support"` in `guides/bug_report_templates/generic_gem.rb`
* Revert "Specify bundler version in template files"SHIBATA Hiroshi2017-12-226-12/+0
| | | | This reverts commit b106242f52272c4a5ced7a0e9d1dcb1b50542501.
* Use released arelyuuji.yaginuma2017-11-156-6/+0
| | | | Related to 2e0fe5928f0d08f85b4796c85bd0b39f6be09079
* Specify bundler version in template filesyuuji.yaginuma2017-11-036-0/+12
| | | | | | | | | | | | We have already specified to install `bundler` 1.15.4 in `.travis.yml`. https://github.com/rails/rails/blob/master/.travis.yml#L31..L32 However, `bundler` 1.16.0 may be used in the test. https://travis-ci.org/rails/rails/jobs/296582467#L2208 The test failed due to this influence. In order to avoid this, specifying `bundler` version in bug report templates.
* Set proper migration' version in bug report templatesbogdanvlviv2017-10-182-2/+2
|
* Deprecate secret_token, long since usurped by secret_key_base.Kasper Timm Hansen2017-09-282-2/+0
| | | | | | | | | | See the changelog entry. Remove `secrets.secret_token` from the bug report templates, since we don't accept bug reports for Rails versions that don't support a `secret_key_base`. [ claudiob & Kasper Timm Hansen ]
* Add newlines before/after the `git_source` in bug report templatesRyuta Kamizono2017-09-2711-0/+22
| | | | [ci skip]
* Change :github git source for bug report templatesbogdanvlviv2017-09-2711-0/+11
| | | | | | | :github source uses `git://` url by default, `https://` is recommended. See http://bundler.io/v1.15/guides/git.html#security We do the same in our `Gemfile` and templates.
* Use frozen string literal in guides/Koichi ITO2017-08-1311-0/+22
|
* Require arel master in the benchmark templateSean Griffin2017-07-251-0/+1
| | | | Pointing at Rails master generally requires pointing at Arel master
* Revert "Merge pull request #29540 from kirs/rubocop-frozen-string"Matthew Draper2017-07-0211-11/+0
| | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa.
* Enforce frozen string in RubocopKir Shatrov2017-07-0111-0/+11
|
* Do not use private API in bug report templatesyuuji.yaginuma2017-06-222-8/+4
| | | | | | | | `ActiveRecord::Migrator` is private API. https://github.com/rails/rails/blob/bb9d6eb094f29bb94ef1f26aa44f145f17b973fe/activerecord/lib/active_record/migration.rb#L977 Therefore, it is not good to use it in bug report templates. Instead, should use the public API `ActiveRecord::Migration#migrate`.
* Define path with __dir__bogdanvlviv2017-05-232-2/+2
| | | | | | ".. with __dir__ we can restore order in the Universe." - by @fxn Related to 5b8738c2df003a96f0e490c43559747618d10f5f
* Update the Gem Bug Report Templates to 5.1.0Alex Kitchens2017-05-015-5/+5
| | | | 5.1.0 has been released, and the gem templates can reflect that now.
* Update bug report templatesMatthew Draper2017-03-225-5/+5
|
* Start Rails 5.2 developmentMatthew Draper2017-03-225-0/+5
|
* Use released arelyuuji.yaginuma2017-02-261-1/+0
| | | | Follow up to ea9566f6cd1b4d3f0d8a5f03283b49423b89044d
* Use released arelRafael Mendonça França2017-02-215-5/+0
|
* Bump bug report template versions to 5.0.1Daniel Martin2017-01-265-5/+5
|
* Remove deprecated support to passing a column to #quoteRafael Mendonça França2016-12-296-0/+6
|
* Add more rubocop rules about whitespacesRafael Mendonça França2016-10-293-4/+4
|
* Introduce a benchmark template [ci skip]Godfrey Chan2016-10-151-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This replaces boilerplate in the “benchmark your code” section of the contributors’ guide with an executable template. I also amended the text to encourage best practices and codified it in the template. For now this is only good for relatively self-contained changes that can be inlined into a simple script. In the future, this can be expanded to cover how to measure the difference between two commits. The output looks like this: ``` ==================================== Empty ===================================== Warming up -------------------------------------- blank? 225.963k i/100ms fast_blank? 238.147k i/100ms Calculating ------------------------------------- blank? 8.825M (± 6.4%) i/s - 44.063M in 5.014824s fast_blank? 9.311M (± 6.3%) i/s - 46.439M in 5.009153s Comparison: fast_blank?: 9310694.8 i/s blank?: 8824801.7 i/s - same-ish: difference falls within error ================================= Single Space ================================= Warming up -------------------------------------- blank? 56.581k i/100ms fast_blank? 232.774k i/100ms Calculating ------------------------------------- blank? 813.985k (±16.7%) i/s - 4.017M in 5.076576s fast_blank? 9.547M (± 5.2%) i/s - 47.719M in 5.013204s Comparison: fast_blank?: 9547414.0 i/s blank?: 813985.0 i/s - 11.73x slower ================================== Two Spaces ================================== Warming up -------------------------------------- blank? 58.265k i/100ms fast_blank? 244.056k i/100ms Calculating ------------------------------------- blank? 823.343k (±16.2%) i/s - 4.020M in 5.014213s fast_blank? 9.484M (± 4.9%) i/s - 47.347M in 5.005339s Comparison: fast_blank?: 9484021.6 i/s blank?: 823343.1 i/s - 11.52x slower =============================== Mixed Whitspaces =============================== Warming up -------------------------------------- blank? 53.919k i/100ms fast_blank? 237.103k i/100ms Calculating ------------------------------------- blank? 763.435k (±16.8%) i/s - 3.720M in 5.018029s fast_blank? 9.672M (± 5.8%) i/s - 48.369M in 5.019356s Comparison: fast_blank?: 9672467.2 i/s blank?: 763435.4 i/s - 12.67x slower =============================== Very Long String =============================== Warming up -------------------------------------- blank? 34.037k i/100ms fast_blank? 240.366k i/100ms Calculating ------------------------------------- blank? 409.731k (± 8.9%) i/s - 2.042M in 5.028235s fast_blank? 9.794M (± 4.3%) i/s - 49.035M in 5.016328s Comparison: fast_blank?: 9794225.2 i/s blank?: 409731.4 i/s - 23.90x slower ```
* bug report template for migrations (#26488)Girish Sonawane2016-09-152-0/+129
| | | * added bug report template for migrations
* fix class name typoyuuji.yaginuma2016-09-022-4/+4
|
* Added bug report templates for Active JobPrathamesh Sonpatki2016-09-022-0/+63
|
* remove unnessary `session_store` settingyuuji.yaginuma2016-08-201-1/+0
| | | | | Since e5a6f7ee9e951dbe0e4e9ea2c0743b4dfb135c57, by default the session store will be set to cookie store with application name as session key.
* applies new string literal convention in guidesXavier Noria2016-08-066-57/+57
| | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* Bump version for master bug report templatesJon Moss2016-07-053-3/+3
|
* Update Ruby report templatesRafael Mendonça França2016-05-103-3/+3
|
* Remove extra space [ci skip]Jon Moss2016-05-051-2/+2
|
* Remove github gems from the master bug report templatesRafael Mendonça França2016-02-183-15/+0
| | | | | All gems are released now so we don't need to ask the github repository for these gems anymore.
* remove backward compatibility code for Minitest 4yuuji.yaginuma2016-01-301-3/+0
| | | | | The master branch is required Ruby 2.2.2+, for the Ruby 2.2 is bundled Minitest 5.4.3, I think backward compatibility code for Minitest 4 is unnecessary.
* Fix: make Travis CI happy about guides againclaudiob2015-09-013-0/+9
| | | | | | | | Tests on Rails [are currently failing](https://travis-ci.org/rails/rails/jobs/78255666). The reason is the dependency of Rails master from gems that are currently on GitHub (not on RubyGems) and should be explicitly referenced in the Guides test files.
* argh!!!Aaron Patterson2015-08-203-0/+3
|
* Replaced render :text with render :plain in AC gem bug report templatePrathamesh Sonpatki2015-07-181-1/+1
| | | | | - Followup of https://github.com/rails/rails/pull/20929. [ci skip]
* use `plain` option instead of deprecated `text` optionyuuji.yaginuma2015-07-181-1/+1
| | | | this will silence deprecation warnings
* Output an error message and raise an error if bundler 1.10.x is not ↵akihiro172015-06-066-6/+36
| | | | installed when reporting bugs
* Add the bug report templates to the Travis CI buildRoque Pinel2015-06-054-9/+24
| | | | | | | | | | | | The bug report templates are now executed from the `ci/travis.rb` when `GEM` contains `guides`. I started by creating a `test` task in `guides/Rakefile` to handle this, but since inline `gemfile` must not be executed with `bundle exec`, that rake task would not be consistent with others. So I went back by executing them directly from `Build`. Use inline Gemfile dependency when reporting gem bugs
* Use inline Gemfile dependency when reporting bugsRoque Pinel2015-06-033-33/+18
| | | | | With Bundler `1.10.3`, it is possible to list the gems inline without the need to create a `Gemfile` if none is found in the current directory.
* Add a "generic" bug report templateGodfrey Chan2015-04-192-0/+41
| | | | | This template gives contributors a starting point to use when reporting bugs that does not involve Active Record or Action Pack.
* load "rack/test" before "action_controller/railtie" in bug report templates ↵yuuji.yaginuma2015-02-051-1/+1
| | | | [ci skip]