| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
y-yagi/remove_backward_compatibility_code_for_minitest_4
remove backward compatibility code for Minitest 4
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
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.
|
| |_|/ /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is to fill in some missing information as apart of #22931.
It's on purpose that the sample `Message` model inherits from
`ActiveRecord::Base` -- Active Record is not meant to be coupled to
Rails, and we can't guarantee that users outside of the Rails world will
have an `ApplicationRecord` class that inherits from
`ActiveRecord::Base`.
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- inbuilt --> built-in
- Remove random spaces from code examples
[ci skip]
|
|\ \ \ \
| | | | |
| | | | | |
Pass 3 over testing guide
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Various grammar fixes
- Added assertions for update controller action tests
- Added user helper tests
- Fix typos
[ci skip]
|
|/ / /
| | |
| | | |
Spelling mistake - direcotry > directory
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Pass through correcting api_app.md. The list of included modules and
middleware was tested through a sample API app, and was listed in the
same order an end user would see in their terminal.
[ci skip]
|
| | |
| | |
| | |
| | |
| | |
| | | |
It is not always there anymore
[ci skip]
|
| | |
| | |
| | |
| | | |
[ci skip]
|
| | |
| | |
| | |
| | | |
The documentation team didn't reviewed it yet so it can't be published.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When I was looking for more info regarding this the only way I ended up
on that page was by googling something along the lines of "rails new
api" (as I wanted to find out what are the proper parameters when
generating api app). I think it's beneficial to have that page in table
of contents.
|
|\ \ \
| | | |
| | | | |
Introduce new welcome page for new projects
|
| | | |
| | | |
| | | |
| | | | |
As requested by David in 23233.
|
| | | |
| | | |
| | | | |
Added an "Or, in English..." explanation to the "Joining Nested Associations (Multiple Level)" example.
|
| | | | |
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Pass 2 over testing guide
|
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Grammar fixes
- Wordsmitting
- Fixed wrong statement about association usage in fixtures
- Changed association name from 'one' to 'first' instead
- More consistent usage of we/our
- Mentions assert_select is below, not already covered in Integration test.
[ci skip]
|
|\ \ \
| | | |
| | | | |
Change permission to readonly [ci skip]
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
changed from 755 to 644.
I executed `chmod -x guides/assets/javascripts/responsive-tables.js`.
[ci skip]
|
|/ /
| |
| |
| |
| | |
Per https://github.com/rails/rails/issues/20264
[ci skip]
|
|\ \ |
|
| | | |
|
| | |
| | |
| | |
| | | |
use bin/rails instead of rake [ci skip]
|
|\ \ \
| | | |
| | | | |
First pass over parts of Testing guide
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Grammar improvements
- Consistent usage of you or we in sentences
- Tests can have zero or more assertions. Its not mandatory to have one.
- Example for assert_send
[ci skip]
|
|/ / / |
|
| | |
| | |
| | |
| | | |
[ci skip]
|
|\ \ \
| | | |
| | | | |
[ci skip] update guide for Puma web server instead of Webrick
|
| | | |
| | | |
| | | | |
Rails 5.0 default server puma web server. following commit - https://github.com/rails/rails/commit/ae48ea69
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
- Only ones left are from the upgrading guide, and command line guide explicit section about rake
Follow up of https://github.com/rails/rails/pull/23119
[ci skip]
|
| | |
| | |
| | | |
I go through the `http://edgeguides.rubyonrails.org/` and found `rake` commands in various files that are in RAILS 5.0 implement by `bin/rails` command. I try to change all that can be directly use `bin/rails …`
|
| | |
| | |
| | |
| | |
| | | |
- Add new logo for guides header
[ci skip]
|
|\ \ \
| | | |
| | | | |
Mention that halting chain does not re-raise `ActiveRecord::RecordIn…
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
exception as well, similar to `ActiveRecord::Rollback`
Fixes #22297
[ci skip]
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
if not marked as safe by using html_safe
Fixes #22648
[ci skip]
|
| | |
| | |
| | |
| | | |
- Followup of https://github.com/rails/rails/commit/71ff088a09d429657877ddfb58985d30df63fc8a
|
| | |
| | |
| | |
| | | |
[ci skip]
|
|\ \ \
| | | |
| | | | |
Changed options for find_each and variants to have options start/finish
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
instead of start_at/end_at based on comments
at https://github.com/rails/rails/pull/12257#issuecomment-74688344
|
| | | |
| | | |
| | | | |
fixes #21500
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
RochesterinNYC/add-order-explanation-to-first-last-guides
Add clarification about `first` and `last` behavior when using `order` [ci skip]
|
| | |/ /
| |/| | |
|
|\ \ \ \
| | | | |
| | | | | |
Updated the guides for having a distinct has_many through at the data…
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The current example
add_index :person_articles, :article, unique: true
Does not work, the `:article` column does not exist as it will be
a join table (based on the prior example) so should use :article_id
The documentation seems to suggest that it will allow an article to
be added only once to a person via the join table, what actually
occurs is that it only allows the article to be added to one person,
at which point it should be a `belongs_to` association.
Also changed the new example to use readings based on the prior example
|
| |_|/ /
|/| | |
| | | |
| | | | |
- Fixes #22876.
|
| | | | |
|