Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix format of generator docs [ci skip] #30259

Merged
merged 1 commit into from
Aug 15, 2017

Conversation

yhirano55
Copy link
Contributor

@yhirano55 yhirano55 commented Aug 15, 2017

Summary

  • Add typewriter style to lib method in actions.rb.
  • Unify typewriter style to ++ in actions.rb.
  • Replace quote style with typewriter style in app_base.rb.

@rails-bot
Copy link

r? @rafaelfranca

(@rails-bot has picked a reviewer for you, use r? to override)

@kamipo
Copy link
Member

kamipo commented Aug 15, 2017

  • Unify typewriter style to ++ in actions.rb.

Why? <tt> doesn't work?

@yhirano55
Copy link
Contributor Author

I thought it would be better that + style is shorter than <tt>.
But it's my mistake... I'll restore it, soon 🙇

@@ -377,7 +377,7 @@ def bundle_command(command)
say_status :run, "bundle #{command}"

# We are going to shell out rather than invoking Bundler::CLI.new(command)
# because `rails new` loads the Thor gem and on the other hand bundler uses
# because +rails new+ loads the Thor gem and on the other hand bundler uses
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually this is a comment in the method, not a doc. It is no worth to change to rdoc format.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fmm.., I understood it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kamipo Thanks for your comment. I'll study more rdoc specifications.

@kamipo kamipo merged commit 24ee261 into rails:master Aug 15, 2017
@yhirano55 yhirano55 deleted the fix_format_of_generator_docs branch August 15, 2017 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants