aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib
Commit message (Collapse)AuthorAgeFilesLines
* Add ActionCable require statement to pluginKoen Punt2016-03-231-2/+4
| | | | | | | | | | When generating a plugin without ActiveRecord (-O), ActionCable wasn't include, which causes problems with the require action_cable statement in cable.js add active_job require statement also updated order of require statements to match with all.rb
* [ci skip] Improve comment minitest plugin initialization.Kasper Timm Hansen2016-03-221-2/+4
| | | | | Didn't feel we were clear enough about our motivation for placing Rails first and why we needed to call `load_plugins`.
* Don't get ahead of ourselves.Kasper Timm Hansen2016-03-221-3/+3
| | | | The useless explicit self calls were driving me nuts.
* Set Rails as the first minitest initialized plugin.Kasper Timm Hansen2016-03-221-1/+3
| | | | | | | | | | | When calling `load_plugins` minitest would fill out its extensions, then we'd tackle ourselves on as the last plugin. Because minitest loads plugins in order we will ultimately have the last say on what reporters will be used. Invert that strategy by putting ourselves first and give other plugins plenty of leeway to override our default reporter setup. Fixes #24179.
* Use || instead of `or` as preferred in rails code conventionMarek L2016-03-222-2/+2
|
* Require enumerable module when using #sumBenjamin Quorning2016-03-221-0/+1
| | | | | | In https://github.com/rails/rails/commit/dfa48f200cbc5c1ca18457a8cde14642e12af5, a call to `#sum` was added. For that to always work, we need to require the core_ext/enumerable module.
* Merge pull request #24253 from y-yagi/change_default_value_of_cachingKasper Timm Hansen2016-03-211-1/+1
|\ | | | | changed default value of `caching` option to `nil`
| * changed default value of `caching` option to `nil`yuuji.yaginuma2016-03-211-1/+1
| | | | | | | | | | | | | | The default is that's false, caching even if you do not specify the caching option is determined not to use, and `tmp/caching-dev.txt` will be deleted. If it is this, regardless of whether or not there is `tmp/caching-dev.txt`, be sure to order would be necessary to specify the caching option, I think that in than good to so as not to do anything by default.
* | Don't upgrade config.file_watcher during upgrade pathGarrett Bjerkhoel2016-03-181-0/+5
|/
* Document and Match the Rails logger interfaceschneems2016-03-161-8/+10
| | | | | | The logger interface to get all Rails features is not obvious. This change adds documentation on how to assign a logger that will use all Rails features. We are also matching the stdout logging interface to the default logger in bootstrap https://github.com/rails/rails/blob/f5a5988352b165143f0f9d622707c351c1470882/railties/lib/rails/application/bootstrap.rb#L42-L45.
* Share dev caching implementation between task and server.Kasper Timm Hansen2016-03-153-25/+46
| | | | | Lets have the server and the Rake task go through the same ropes, so we can share all the file lookup, adding, and deleting code.
* Remove unfinished command infrastructure.Kasper Timm Hansen2016-03-102-72/+2
| | | | | | | If we're gonna do this right, it will look mighty different from this anyway. (Looking at you, Rails 5.1). It isn't being used in any code as of now, so yanking is the best option.
* prefer require_relative over require + File.expand_pathXavier Noria2016-03-101-1/+1
|
* Merge pull request #23940 from prathamesh-sonpatki/rm-with-dispatchersRafael França2016-03-091-2/+3
|\ | | | | Remove passing `with_dispatchers` to AppGenerator
| * Remove passing `with_dispatchers` to AppGeneratorPrathamesh Sonpatki2016-03-091-2/+3
| | | | | | | | | | | | | | - While working on https://github.com/rails/rails/pull/22790 I found out that we pass `with_dispatchers: true` but it's not used at all by Rails. - We were passing `with_dispatchers: true` in few tests too but again it's not used by Rails.
* | AppGenerator: allow both 'rake' and 'rails'claudiob2016-03-091-6/+21
| | | | | | | | | | | | | | | | | | This commit comes from the comments made by @matthewd at https://github.com/rails/rails/pull/23795/files#r54469637 and by @rafaelfranca at https://github.com/rails/rails/pull/23795/files#r54609364 The idea is that if you type (for example) "rake db:migrate" in an AppGenerator, then this should actually invoke `rake db:migrate` on the command line, whereas if you type "rails_command db:migrate", this should invoke `rails db:migrate`.
* | Merge pull request #24062 from resure/redundant-escapesRafael França2016-03-082-2/+2
|\ \ | | | | | | Remove redundant regexp escapes in generators
| * | Remove redundant regexp escapes in generatorsGadzhi Gadzhiev2016-03-082-2/+2
| | |
* | | Advertise the console web-console featureGenadi Samokovarov2016-03-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | In the latest version of Web Console, you can call `console` anywhere in your application and get a console with that binding in it. I think this is worth advertising, as it may be useful to do outside of controllers and views. [ci skip]
* | | Drop the explicit version requirements for web-consoleGenadi Samokovarov2016-03-081-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | Between major versions 2 and 3, we hit a bug. It's fixed in version 3, however, the explicit 2.x requirement of `~> '2.0'` will prevent people from getting the fix with `bundle update` and they would have to explicitly set the constraint to `~> '3.0'`. For more information see: rails/web-console#178. I propose we drop the explicit version constraints in the Gemfile. Web Console has been relatively stable for the past couple of years, and I don't anticipate any major alterations, like we saw between major versions 1 and 2.
* | prefer require_relative over require + File.expand_pathXavier Noria2016-03-064-4/+5
| | | | | | | | Reads much better doesn't it.
* | use __dir__ for relative paths in config/application.rbXavier Noria2016-03-062-2/+2
| | | | | | | | | | | | Stuff I see in passing while I am doing a walkthrough. See rationale in 5b8738c.
* | use __dir__ for relative paths in config/boot.rbXavier Noria2016-03-062-3/+3
| | | | | | | | | | | | Stuff I see in passing while I am doing a walkthrough. See rationale in 5b8738c.
* | use __dir__ to load the application in environment.rbXavier Noria2016-03-061-1/+1
| | | | | | | | | | | | Stuff I see in passing while I am doing a walkthrough. See rationale in 5b8738c.
* | use __dir__ to load the environment in config.ruXavier Noria2016-03-061-1/+1
| | | | | | | | | | | | Stuff I see in passing while I am doing a walkthrough. See rationale in 5b8738c.
* | Merge pull request #24070 from ↵Rafael França2016-03-052-13/+0
|\ \ | | | | | | | | | | | | y-yagi/remove_unnecessary_config_in_sprockets-rails-3 remove config that are no longer needed in sprockets-rails 3
| * | remove config that are no longer needed in sprockets-rails 3yuuji.yaginuma2016-03-052-13/+0
| | | | | | | | | | | | | | | `config.assets.raise_runtime_errors` and `config.assets.digest` are enabled by default in sprockets-rails 3.
* | | define APP_PATH with __dir__Xavier Noria2016-03-051-1/+1
|/ / | | | | | | | | Defining paths relative to a file name has always hurt my soul, with __dir__ we can restore order in the Universe.
* | [ci skip] correcting sentence in description of application.rbMohit Natoo2016-03-041-2/+1
| | | | | | | | Adding period in the end.
* | Fix description in cookies_serializer.rb being corrupted when updatingHirofumi Wakasugi2016-03-021-1/+1
| |
* | Merge pull request #23935 from y-yagi/convert_cable_coffee_to_javascriptRafael França2016-03-013-10/+14
|\ \ | | | | | | convert cable.coffee to cable.js
| * | convert cable.coffee to cable.jsyuuji.yaginuma2016-02-283-10/+14
| |/ | | | | | | In order to eliminate the dependecy of CoffeeScript.
* | Merge pull request #23981 from y-yagi/do_not_set_action_cable_configRafael França2016-03-011-4/+3
|\ \ | | | | | | do not set Action Cable's config when specify `--skip-action-cable` option
| * | do not set Action Cable's config when specify `--skip-action-cable` optionyuuji.yaginuma2016-03-011-4/+3
| | | | | | | | | | | | Also remove duplicated `config.action_cable.url` config.
* | | Merge pull request #23986 from rubys/dedup-ctrlc-msgEileen M. Uchitelle2016-03-011-2/+0
|\ \ \ | | | | | | | | Delete CTRL-C message as is duplicates Puma
| * | | Delete CTRL-C message as is duplicates PumaSam Ruby2016-03-011-2/+0
| |/ /
* | | Use AS::Executor / AS::Reloader to support reloading in ActionCableMatthew Draper2016-03-021-6/+0
| | |
* | | Publish AS::Executor and AS::Reloader APIsMatthew Draper2016-03-024-35/+69
|/ / | | | | | | | | | | These should allow external code to run blocks of user code to do "work", at a similar unit size to a web request, without needing to get intimate with ActionDipatch.
* | generate config/spring.rb in new applications [closes #18874]Xavier Noria2016-02-292-0/+7
| |
* | Merge pull request #23929 from ↵Kasper Timm Hansen2016-02-281-2/+2
|\ \ | |/ |/| | | | | prathamesh-sonpatki/update-deprecation-message-for-app-namespace Update deprecation message shown when tasks from rails namespace are run
| * Update deprecation message shown when tasks from rails namespace are runPrathamesh Sonpatki2016-02-271-2/+2
| |
* | Merge pull request #23921 from y-yagi/remove_rack_cors_initializer_when_updatingArthur Nogueira Neves2016-02-271-0/+5
|\ \ | |/ |/| remove rack cors initializer when updating
| * remove rack cors initializer when updatingyuuji.yaginuma2016-02-271-0/+5
| | | | | | | | Rack cors initializer is only necessary to API-only applications, for when the update is unnecessary.
* | Merge pull request #23439 from ryohashimoto/160203_rake_railsKasper Timm Hansen2016-02-272-2/+16
|\ \ | | | | | | Use app namespace for framework tasks
| * | The tasks in the rails task namespace is deprecated in favor of app namespace.Ryo Hashimoto2016-02-262-2/+16
| | | | | | | | | | | | (e.g. `rails:update` and `rails:template` tasks is renamed to `app:update` and `app:template`.)
* | | :scissors:Rafael Mendonça França2016-02-261-2/+2
| | |
* | | Merge pull request #23917 from chashmeetsingh/patch-1Rafael Mendonça França2016-02-261-1/+3
|\ \ \ | | | | | | | | | | | | Create tmp folder if not present
| * | | Create tmp folder if not presentChashmeet Singh2016-02-271-1/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All Apps may not have a tmp folder Update dev.rake Update dev.rake Update dev.rake Update dev.rake Update dev.rake Update dev.rake Update dev.rake
* / | Specify plaform: :mri for byebugIan Ker-Seymer2016-02-261-1/+1
|/ /
* / [close #23681] Use puma 3.0.0+ schneems2016-02-261-1/+1
|/ | | | | Puma 3.0 and up introduced compatibility to read from `config/puma.rb` when booting from the command `$ rails server`https://github.com/puma/puma/pull/856.
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-06 19:19:10 +0000