- 21 10月, 2018 5 次提交
-
-
由 Eugene Kenny 提交于
When `attr_name` is passed as a symbol, it's currently converted to a string by `attribute_alias?`, and potentially also `attribute_alias`, as well as by the `read_attribute`/`write_attribute` method itself. By converting `attr_name` to a string up front, the extra allocations related to attribute aliases can be avoided.
-
由 Ryuta Kamizono 提交于
Add message to exception in datetime type
-
由 wilddima 提交于
-
由 Francesco Rodríguez 提交于
-
由 Jeremy Daer 提交于
Improve layout of Rails Guides index
-
- 20 10月, 2018 13 次提交
-
-
由 Ryuta Kamizono 提交于
Since #29294, `mattr_acessor` uses kwargs for `instance_reader`, `instance_writer`, and `instance_accessor` options. `thread_mattr_accessor` and `config_accessor` also take the same options, so let's maintain these options handles the same.
-
由 Ryuta Kamizono 提交于
Remove unnecessary escape character
-
由 Akira Matsuda 提交于
-
由 Akira Matsuda 提交于
-
由 Akira Matsuda 提交于
-
由 Akira Matsuda 提交于
`ruby -ractive_support/core_ext/range/conversions.rb -ep` dies with uninitialized constant ActiveSupport
-
由 Akira Matsuda 提交于
-
由 Akira Matsuda 提交于
-
由 yuuji.yaginuma 提交于
If use `run_generator` to run the generator, `--skip-webpack-install` is specified automatically. https://github.com/rails/rails/blob/3101a4136bd62787e252d2658eee23001036fa0f/railties/lib/rails/generators/testing/behaviour.rb#L71 However, when executing the generator independently (for example, to use stub), `webpacker:install` was executed. Since this includes `yarn install`, it should be avoided in unnecessary testing.
-
由 Olivier Lacan 提交于
The previous version wasn’t matching the Rails Guides logo source file listed at https://github.com/rails/homepage/blob/master/materials/rails_guides_logo.psd somehow. It was also a GIF when an 8-bit PNG results in a smaller file (2KB vs 4KB) which should be just as widely compatible with browsers. And that’s just for the 1x version. I added a new 2x version for “retina” or high-density displays and a CSS @media query to progressively enhance with this high density version of the logo. I’ve ensured that this query is supported across browsers by using vendor-specific versions when necessary. See: https://caniuse.com/#feat=css-media-resolution [ci skip]
-
由 Olivier Lacan 提交于
The names are only used in the Guides Index overlay and several of them are longer than they need to be. For example “Ruby on Rails” is mentioned over and over again in the Release Notes section although these are obviously all Rails versions we’re listing. There’s a Maintenance Policy category with a Maintenance Policy article which is redundant. Policies makes much more sense as a category. Similarly, Contributing to Ruby on Rails is both a category and an article. “Contributions” is a better category title. “Ruby on Rails Guides Guidelines” is a gigantic mouthful. We are *in* the Rails Guides, we don’t need to specify what guides we’re talking about here. As a final added advantage, all of these shortenings make the index much easier to parse in order to find an article. [ci skip]
-
由 Olivier Lacan 提交于
This notably allows us to tile each category without pushing the page down vertically as much as we previously did on any viewport wider than mobile sizes. It also means we can fit more guides which will become useful in the future since we have several new guides in the work. Finally the new layout allows three clearly distinct columns to emerge: - Start Here and Rails basics: Models, Views, Controllers, Other Components - Digging Deeper, Extending Rails, Contributing, Maintenance - Release Notes Having Release Notes shoot back up to the top of the third column is great because that makes finding the latest released version and its associated release notes much easier without having to scroll down a bunch. [ci skip]
-
由 Richard Schneeman 提交于
Add guides section on verbose query logs to Debugging
-
- 19 10月, 2018 9 次提交
-
-
由 Olivier Lacan 提交于
Since this is a useful tool in debugging it made sense to document its existence and usage, especially in the console where it's disabled by default. [ci skip]
-
由 Francesco Rodríguez 提交于
-
由 yuuji.yaginuma 提交于
If the dev option is specified, Gemfile contains gem which specifies GitHub. This will take time to execute, so should avoid it in unnecessary tests.
-
由 Yuji Yaginuma 提交于
Make AJ integration tests work in CI
-
由 yuuji.yaginuma 提交于
Since `@mutex.synchronize` is enforced in the `ensure_connection!` method, there is no need to do so on the caller side. https://github.com/jondot/sneakers/blob/c1b47f9c5d5a95da728bbe1700795790e4efbb12/lib/sneakers/publisher.rb#L22-L26 Due to this, `ThreadError(deadlock; recursive locking)` has occurred.
-
由 yuuji.yaginuma 提交于
Also, replace the authentication setting. This is necessary for using `psql` with PostgreSQL 10. Ref: https://github.com/travis-ci/travis-ci/issues/9624#issuecomment-389537036
-
由 Yuji Yaginuma 提交于
Fix typo in testing guide
-
由 yuuji.yaginuma 提交于
-
由 Francesco Rodríguez 提交于
in favor of `array.flatten.pack("U*")` and `string.scan(/\X/).map(&:codepoints)`, respectively.
-
- 18 10月, 2018 3 次提交
-
-
由 Yuji Yaginuma 提交于
Fix generated Gemfile missing gems on jruby
-
由 anthonygharvey 提交于
-
由 Ryuta Kamizono 提交于
-
- 17 10月, 2018 10 次提交
-
-
由 David Rodríguez 提交于
-
由 Richard Schneeman 提交于
ActiveRecord#respond_to? No longer allocates strings
-
由 Ryuta Kamizono 提交于
Follow up #32146.
-
由 Ryuta Kamizono 提交于
Remove index:true option from belongs to as defaults to true. [ci skip]
-
由 Yuji Yaginuma 提交于
Update guide for the counter variable when rendering with the `as:` option
-
由 Ryuta Kamizono 提交于
`init_with` and `init_from_db` are almost the same code except decode `coder`. And also, named `init_from_db` is a little misreading, a raw values hash from the database is already converted to an attributes object by `attributes_builder.build_from_database`, so passed `attributes` in that method is just an attributes object. I renamed that method to `init_with_attributes` since the method is shared with `init_with` to initialize an empty model object.
-
由 Lucas Oliveira 提交于
[ci skip]
-
由 Ryuta Kamizono 提交于
Consistently extract checking version for all adapters
-
由 Francesco Rodríguez 提交于
Use \X meta character directly to get grapheme clusters. Thanks to @mtsmfm for the tip: https://github.com/rails/rails/pull/34123#issuecomment-429028878 r? @jeremy
-
由 Ryuta Kamizono 提交于
I don't prefer to extract it for one adapter even though all adapters also does. Related to #34227.
-