aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib/rails/generators
Commit message (Collapse)AuthorAgeFilesLines
* require object/blank not usedAnkit Gupta2015-05-061-1/+0
| | | | the current class is not the blank?, present? as well as other inheriting, the test suite runs
* Merge pull request #20023 from mohnish/patch-1Yves Senn2015-05-061-2/+1
|\ | | | | | | Use the built-in rake tasks
| * Use the built-in rake tasksMohnish Thallavajhula2015-05-041-2/+2
| | | | | | Use the `log:clear` and `tmp:clear` rake tasks in the setup file template
* | list reserved words in generator errorSean Linsley2015-05-052-2/+2
|/
* Removed `assigns` from functional_test templatesPrathamesh Sonpatki2015-05-021-3/+2
| | | | | - Based on DHH's suggestion about deprecating `assigns` in https://github.com/rails/rails/pull/18305#issuecomment-68605166.
* Escape README in Plugin builder API documentationPrathamesh Sonpatki2015-04-261-1/+1
| | | | | - So that RDoc will not generate link for README. [ci skip]
* Do not show --helper and --assets options in the controller generator helpAkira Matsuda2015-04-251-1/+2
| | | | | These options are not actually available. Thanks @soutaro for letting me aware of this.
* Merge pull request #19858 from cllns/fix-scaffolding-css-formattingRafael Mendonça França2015-04-221-7/+20
|\ | | | | Fix formatting of scaffold css
| * Fix formatting of scaffold cssSean Collins2015-04-221-7/+20
| |
* | Add blank lines between divs in form scaffoldingSean Collins2015-04-201-0/+2
|/
* Implicitly skip spring for `rails new --dev`Godfrey Chan2015-04-201-1/+1
| | | | | | Spring seems like an obvious foot gun for the `--dev` setup, presumably you are about to make lots of changes to the codebase, so it should always reload the framework.
* fix indent in routes when using namespaced controllersyuuji.yaginuma2015-04-141-3/+4
| | | | | | | | | | | | | | | | | | Before: namespace :foo do namespace :bar do get 'baz/index' end end After: namespace :foo do namespace :bar do get 'baz/index' end end
* Avoid to define an initializer after the load_config_initializersRafael Mendonça França2015-04-071-1/+1
| | | | | This make the config/initializers run before the railties are loaded what can break some configurations.
* Merge pull request #19429 from mxhold/print_bundle_install_output_line_by_lineMatthew Draper2015-04-031-6/+6
|\ | | | | | | Print `bundle install` output in `rails new` as soon as it's available
| * Print `bundle install` output in `rails new` as soon as it's availableMax Holder2015-03-281-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, running `rails new` would not print any of the output from `bundle install` until all the gems had finished installing. This made it look like the generator was hanging at the `bundle install` step. This commit switches to using `system` so that the bundle command can output as it needs to. This has the added benefit of including output bundler produces on standard error, which the previous code ignored since backticks only capture standard out. This is not a big deal right now since bundler does not currently print errors to standard error, but that may change in the future (see: bundler/bundler/issues/3353).
* | remove extra newline from ApplicationJob templateyuuji.yaginuma2015-03-311-1/+0
| |
* | Merge pull request #19034 from jvanbaarsen/explicit-job-base-classMatthew Draper2015-03-301-0/+3
|\ \ | | | | | | | | | Add explicit base class for ActiveJob jobs
| * | Add explicit base class for ActiveJob jobsJeroen van Baarsen2015-03-231-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | * Jobs generated now inherent from ApplicationJob * ApplicationJob inherents from ActiveJob::Base * Added entry to changelog Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
* | | respect `pluralize_table_names` when generate fixture file. fixes #19519yuuji.yaginuma2015-03-272-1/+5
| |/ |/|
* | Updated MySQL documentation link to MySQL latest version 5.6 everywhere [ci ↵amitkumarsuroliya2015-03-192-2/+2
| | | | | | | | skip]
* | method_source needs to be a hard dependency on railtiesArthur Neves2015-03-191-4/+0
| |
* | Add a new-line to the end of route method generated code.Arthur Neves2015-03-182-3/+5
|/ | | | | | | | Fix regression on route method that was added by bac812a7ef2660a2fe2ab00822e5e66228379822. The regression was that when calling the `route` method, we were not appending a \n anymore. [fixes #19316]
* add mailer suffix to comments for email previewsyuuji.yaginuma2015-03-151-2/+2
|
* Update sprockets links to point to rails orgJoshua Peek2015-03-133-3/+3
|
* Require `belongs_to` by default.Josef Šimánek2015-02-212-0/+15
| | | | Deprecate `required` option in favor of `optional` for belongs_to.
* Prefer request_id over uuid and test the aliasRafael Mendonça França2015-02-201-1/+1
|
* Remove last empty line when we don't use Windows or JRubymeganemura2015-02-202-2/+2
|
* Merge pull request #18393 from y-yagi/fix_mailerRafael Mendonça França2015-02-183-2/+11
|\ | | | | follow up to #18074
| * follow up to #18074yuuji.yaginuma2015-01-083-2/+11
| | | | | | | | | | * ignore mailer suffix when generate test files * add mailer suffix to view files
* | Revert "Merge pull request #18951 from ↵Rafael Mendonça França2015-02-181-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | jurezove/gemfile-lock-to-plugin-gitignore-template" This reverts commit b824dfd44bf2dd932d6d955ac868af1020f0099a, reversing changes made to 265dcb779dd4bfd9745e0131f585749d7034c3c7. Our general recommendation is to check in the lock file in the repository. This make possible to developers and contributors of a plugin always run their tests against a working set of dependencies. Also, bisect is a lot easier, specially if you have git dependencies in your Gemfile.
* | Add Gemfile.lock to plugin's gitignore templateJure Žove2015-02-151-0/+1
| |
* | Remove warning from Plugin GeneratorYuki Nishijima2015-02-081-3/+3
| | | | | | | | | | | | This removes the following warning: rails/railties/lib/rails/generators/rails/plugin/plugin_generator.rb:321: warning: shadowing outer local variable - content
* | README.rdoc -> README.md for newly generated applicationsXavier Noria2015-02-062-2/+2
| | | | | | | | | | | | README.rdoc was generated to support the doc:app task. Now that this task is gone we can switch to Markdown, which is nowadays a better default.
* | Remove documentation tasksXavier Noria2015-02-062-10/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch removes the tasks doc:app, doc:rails, and doc:guides. In our experience applications do not generate APIs using doc:app. Methods may be certainly documented for maintainers, annotated with YARD tags, etc. but that is intended to be read with the source code, not in a separate website. Then, teams also have typically selected topics written down in Markdown files, or in a GitHub wiki... that kind of thing. If a team absolutely needs to generate application documentation for internal purposes, they can still easily write their own task. Regarding doc:rails and doc:guides, we live in 2015. We are used to go to online docs all the time. If you really want access to the API offline RubyGems generates it for every Rails component unless you tell it not to, and you can checkout the Rails source code to read the guides as Markdown, or download them for a Kindle reader. All in all, maintaining this code does not seem to be worthwhile anymore. As a consequence of this, guides (+3 MB uncompressed) won't be distributed with the rails gem anymore. Of course, guides and API are going to be still part of releases, since documentation is maintained alongside code and tests. Also, time permitting, this will allow us to experiment with novel ways to generate documentation in the Rails docs server, since right now we were constrained by being able to generate them in the user's environment.
* | Merge pull request #18526 from vipulnsward/add-silence-streamRafael Mendonça França2015-02-051-16/+2
|\ \ | | | | | | Extracted silence_stream method to new module in activesupport/testing
| * | - Extracted silence_stream method to new module in activesupport/testing.Vipul A M2015-01-201-16/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added include for the same in ActiveSupport::Test. - Removed occurrences of silence_stream being used elsewhere. - Reordered activesupport testcase requires alphabetically. - Removed require of silence stream from test_case - Moved quietly method to stream helper - Moved capture output to stream helper module and setup requires for the same elsewhere
* | | Force generated route to be insertedAndrew White2015-02-051-1/+1
| | | | | | | | | | | | | | | | | | | | | Thor isn't very discerning over whether some content is present when passed to `inject_into_file`, e.g. a commented out route is detected as being present. So to prevent people scratching their heads as to why a route hasn't appeared it's better to fall on the side of having duplicate routes.
* | | Don't remove all line endings from routes.rbAndrew White2015-02-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When there is a single scaffold in the routes.rb with no other lines then revoking/destroying it will create a routes.rb file with a syntax error. This is because the sentinel for the Thor `route` action didn't include the newline but the logged route code did. The fix is to add the newline to the sentinel and remove it from the the logged route code. Fixes #15913.
* | | Use public Module#include, in favor of https://bugs.ruby-lang.org/issues/8846robertomiranda2015-01-312-2/+2
| | | | | | | | | | | | ref: https://github.com/rails/rails/pull/18763#issuecomment-72349769
* | | Fixed a few things from ee9e4c3Godfrey Chan2015-01-304-5/+5
| | | | | | | | | | | | | | | Mostly cosmetics, except that `isolate_namespace` should be applied to the last module in the chain.
* | | Allow hyphenated names for `rails plugin new` generator.Robin Böning2015-01-3020-63/+92
| | | | | | | | | | | | | | | | | | It was not possible to create a new gem with a hyphenated name via the `rails plugin new` generator. The naming guide of rubygems clearly says dashes should be used for gems that extend other gems. http://guides.rubygems.org/name-your-gem/
* | | scaffold controller_test template should use kwargs. refs #18323.Yves Senn2015-01-301-5/+5
| | | | | | | | | | | | This prevents a flood of warnings when generating a new scaffold.
* | | Add a margin to body and padding to th, td.Adrien Lamothe2015-01-291-0/+11
| | | | | | | | | | | | Add space to new css defs.
* | | Merge pull request #18705 from mrgilman/skip-testRafael Mendonça França2015-01-297-11/+11
|\ \ \ | | | | | | | | Update option to skip test in generators
| * | | Change option description and update test namesMelanie Gilman2015-01-291-1/+1
| | | |
| * | | Update option to skip test in generatorsMelanie Gilman2015-01-277-11/+11
| | | | | | | | | | | | | | | | Rails no longer generates Test::Unit files by default.
* | | | Merge pull request #18719 from ↵Yves Senn2015-01-291-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5t111111/add-fix-for-loading-fixtures-in-engine-tests Add fix for loading fixtures in engine tests (additional fix for #4971) Conflicts: railties/lib/rails/generators/rails/plugin/templates/test/test_helper.rb
| * | | | Add fix for loading fixtures in engine testsWAKASUGI 5T1111112015-01-291-0/+1
| | | | |
* | | | | reassign `file_fixture_path` in plugins `test_helper.rb`. refs #18658.Yves Senn2015-01-291-0/+1
|/ / / /
* | | | table name should be plural [ci skip]Jian Weihang2015-01-291-1/+1
| | | |