{"push_id":652421579,"size":1000,"distinct_size":58,"ref":"refs/heads/master","head":"21c74bd769f6c873453e9244b0de7ced40a532be","before":"bb0518891c4c7cc40ff01d90983b83beb61242af","commits":[{"sha":"c7172f50408cc29f22fa5be43f54ffa0dc43657b","author":{"email":"6c63aa6d24bf282c6bc4bc8e8571a207c1c35843@gmail.com","name":"Kevin Jayanthan"},"message":"Fixing incorrect class name used in an example.\n\nOn \"Controller Namespaces and Routing\" section of the \"Rails Routing from the Outside In\" , the examples are using \"AritclesController\" as an example of a controller, to show different organizers. But on the line 232, it incorrectly uses \"PostsController\" instead of \"AritclesController\".","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/c7172f50408cc29f22fa5be43f54ffa0dc43657b"},{"sha":"ec8d8040a9c2d63c34ce0d3d3186c6fbb4230f37","author":{"email":"74483688a4235936f7a77c2cc60f0c2006cf1bda@gmail.com","name":"Arun Agrawal"},"message":"Merge pull request #19167 from limekin/master\n\nOn guides, correcting incorrect controller name used on routing.md.","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/ec8d8040a9c2d63c34ce0d3d3186c6fbb4230f37"},{"sha":"f46b198142d2e4f436dae4983aff667a0f352b4a","author":{"email":"4df7f3dad182da66af235a1249daf95d26b3dee6@gmail.com","name":"Genadi Samokovarov"},"message":"Web Console documentation for the upcoming 2.1 release [ci skip]\n\nWith the upcoming 2.1 [web-console release], I have tweaked the existing\ndocumentation a bit. I tried to focus on the spawning console use case,\nbecause I think a lot of people overlook that.\n\nTried to explain it as best as I can, however, my English and prose\naren't my best skills :) If you guys can chime in and help me improve\nthe wording, I would be extremely grateful.\n\nThere has been some configuration default changes. For example, the\nwhitelisted IPs always include IPv4 and IPv6 localhosts now and this\nwasn't the case in 2.0. I think a lot of people got bitten by it, that's\nwhy I changed it. I'm a bit confused on how to document this. Should I\njust document the latest version, I don't think I can expect all the\npeople reading the guide to be on it.\n\n[web-console release]: https://github.com/rails/web-console/pull/110","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/f46b198142d2e4f436dae4983aff667a0f352b4a"},{"sha":"435aa7e34f411ea9802345da9a1cfa04bd4deabf","author":{"email":"f10392fd354b3cbb5c499dcfbcd43650305bc398@gmail.com","name":"Anton Davydov"},"message":"[ci skip] Add code example for MailHelper#block_format documentation","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/435aa7e34f411ea9802345da9a1cfa04bd4deabf"},{"sha":"62dfb16842b86a27fb98285958eea075561d4a66","author":{"email":"4df7f3dad182da66af235a1249daf95d26b3dee6@gmail.com","name":"Genadi Samokovarov"},"message":"Nit pick the byebug settings guide for default values [ci skip]\n\nIn #19097 we had a discussion where we decided to change `byebug`\nsettings sections default part to actually say `defaults`.","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/62dfb16842b86a27fb98285958eea075561d4a66"},{"sha":"76e37e6fa8a98a5b4cf389cf18c889c41520da1f","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Merge pull request #19097 from gsamokovarov/web-console-guide\n\nWeb Console documentation for the upcoming 2.1 release [ci skip]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/76e37e6fa8a98a5b4cf389cf18c889c41520da1f"},{"sha":"565bf8926e48151cb3168073132bc6016f46b32e","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Merge pull request #19168 from gsamokovarov/byebug-defaults-guide\n\nNit pick the byebug settings guide for default values [ci skip]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/565bf8926e48151cb3168073132bc6016f46b32e"},{"sha":"9f4210cd80f452685b44c198d5a361ee2b54c05e","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Merge pull request #19133 from davydovanton/update-doc-example-mail-block-format\n\n[ci skip] Add code example for MailHelper#block_format documentation","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/9f4210cd80f452685b44c198d5a361ee2b54c05e"},{"sha":"022c8be2cfee97b76c38e5134aa223f62938d6bf","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Avoid parallelizing Action Pack's tests on Rubinius\n\nSince the `ForkingExecutor` class seems to be pretty slow on Rubinius\ndue to DRb (c.f. http://git.io/xIVg), let's avoid running tests with\nit on this platform.\n\nAlso, the `parallelize_me!` call make the suite to output a bunch of\nerrors due to rubinius/rubinius#2934 since there are thread-safety\nproblems with autoloading.","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/022c8be2cfee97b76c38e5134aa223f62938d6bf"},{"sha":"586fe4471d3a71b7500f734d26b28427332b6e43","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Avoid accurate assertions on error messages\n\nSince there are disparities between the raised error messages on the\ndifferent implementations, let's avoid being too accurate.","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/586fe4471d3a71b7500f734d26b28427332b6e43"},{"sha":"823e4e91158a7e50a56fb92519fc50917b3b33fd","author":{"email":"73e88645cae6486da78d02f3a03ed68d9cb5f050@gmail.com","name":"Robin Dupret"},"message":"Add the platform-specific skip helpers to ActiveModel","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/823e4e91158a7e50a56fb92519fc50917b3b33fd"},{"sha":"b7aa845c018f3b01e70f274a9b30112276a6dcaa","author":{"email":"edede0f6f2949ae2e20b736f5dbb38abbe2fb229@gmail.com","name":"Yu Haidong"},"message":"Modify guides for a better sounding sentence","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/b7aa845c018f3b01e70f274a9b30112276a6dcaa"},{"sha":"f08c3f737b7e77a5069dbef9c591c89d163b75d2","author":{"email":"7a0559213618bb59f33e95efd260811728549345@gmail.com","name":"Eileen M. Uchitelle"},"message":"Merge pull request #19106 from teeceepee/patch-1\n\nImprove guides [ci skip]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/f08c3f737b7e77a5069dbef9c591c89d163b75d2"},{"sha":"2675a57fc3071c49ba8276c50f29dde3d427bc7f","author":{"email":"84c66fc058fe65d1bb08ca33b21bfe0fa5cb8f17@gmail.com","name":"Rafael Mendonça França"},"message":"Do not output in the console\n\n[Robin Dupret + Rafael Mendonça França]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/2675a57fc3071c49ba8276c50f29dde3d427bc7f"},{"sha":"beaf8cbad997fa5c05e309eea6ad1ed8f414f385","author":{"email":"0429096870cbcfd873d290532ef3e340a4b693aa@columbia.edu","name":"Hyonjee Joo"},"message":"Rake restart task no longer loads entire Rails environment when run.\nThe restart task does not need access to models or other classes and\nhelpers from the application environment.","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/beaf8cbad997fa5c05e309eea6ad1ed8f414f385"},{"sha":"9856baf0d3c119f3c938742afe4b9297bd9633e8","author":{"email":"b513427cf64f56e62862790db6873c548db0445e@gmail.com","name":"Juanito Fatas"},"message":"Clarify that t.references and t.belongs_to are interchangeable. [ci skip]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/9856baf0d3c119f3c938742afe4b9297bd9633e8"},{"sha":"230393a5bdf10089006b7666fa15cb4543a46486","author":{"email":"7a0559213618bb59f33e95efd260811728549345@gmail.com","name":"Eileen M. Uchitelle"},"message":"Merge pull request #19169 from hjoo/rake_restart\n\nRake restart task no longer loads entire Rails environment when run","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/230393a5bdf10089006b7666fa15cb4543a46486"},{"sha":"10058ea2805e2347fe070b937c62c878269d1b1d","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"remove useless instance variable\n\ndepth is always 0, so the index will always be false.  No reason to\ncreate the instance variable if it isn't used","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/10058ea2805e2347fe070b937c62c878269d1b1d"},{"sha":"57d35b2bf9e48173a5f97ccff5e6897f0c46411f","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"call `sync_with_transaction_state` inside `persisted?` then check ivars\ndirectly\n\ncalling `sync_with_transaction_state` is not fast, so if we call it\nonce, we can improve the performance of the `persisted?` method.  This\nis important because every call to `url_for(model)` will call\n`persisted?`, so we want that to be fast.\n\nHere is the benchmark:\n\n```ruby\nrequire 'active_record'\n\nActiveRecord::Base.establish_connection adapter: \"sqlite3\", database: \":memory:\"\n\nActiveRecord::Base.connection.instance_eval do\n  create_table(:articles)\nend\n\nclass Article < ActiveRecord::Base; end\narticle = Article.new.tap(&:save!)\n\nBenchmark.ips do |x|\n  x.report(\"persisted?\") do\n    article.persisted?\n  end\nend\n```\n\nBefore this patch:\n\n```\n$ bundle exec ruby -rbenchmark/ips persisted.rb\nCalculating -------------------------------------\n          persisted?     3.333k i/100ms\n-------------------------------------------------\n          persisted?     51.037k (± 8.2%) i/s -    253.308k\n```\n\nAfter:\n\n```\n$ bundle exec ruby -rbenchmark/ips persisted.rb\nCalculating -------------------------------------\n          persisted?     7.172k i/100ms\n-------------------------------------------------\n          persisted?    120.730k (± 5.1%) i/s -    602.448k\n```","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/57d35b2bf9e48173a5f97ccff5e6897f0c46411f"},{"sha":"f9ed46bf5071545cce629cc6164029906cb2eb36","author":{"email":"999e8c50cd8b45fcffd2276fb62f876489a73119@gmail.com","name":"Carlos Antonio da Silva"},"message":"Merge pull request #19170 from JuanitoFatas/doc/active_record/connection_adapters/abstract/schema_definitions.rb\n\nClarify that t.references and t.belongs_to are interchangeable. [ci skip]","distinct":false,"url":"https://api.github.com/repos/zamith/rails/commits/f9ed46bf5071545cce629cc6164029906cb2eb36"}]}