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

Add --skip-action-mailer and change the generators to pick the wanted frameworks in the Gemfile #5703

Closed
wants to merge 7 commits into from

Conversation

rafaelfranca
Copy link
Member

This pull request change the application generator to pick the frameworks in the Gemfile instead of in the application.rb

This is related with #5652

@ahawkins
Copy link

+1

@steveklabnik
Copy link
Member

This no longer merges cleanly, and will need a rebase.

@frodsan
Copy link
Contributor

frodsan commented Jun 23, 2012

@rafaelfranca any news?

@rafaelfranca
Copy link
Member Author

Not yet. I'll close this pull request since it will not be accept as it is.

@claudiob
Copy link
Member

@rafaelfranca I was thinking about suggesting a --skip-action-mailer option for the Rails app generator, and I saw this Pull Request, which was opened and closed by you.

Was there a specific reason why it was closed? Would you accept a new PR introducing a --skip-action-mailer option?

@rafaelfranca
Copy link
Member Author

Please go ahead. This PR also changed the way frameworks are pick, so this was the reason I closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants