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

Rename all .rb templates which contain erb to .rb.erb. #1750

Merged
merged 2 commits into from
Jun 20, 2012
Merged

Conversation

ugisozols
Copy link
Member

@@ -7,8 +7,6 @@ def setup_environment
elsif File.dirname(__FILE__) =~ %r{vendor/extensions}
# Require the path to the refinerycms application this is vendored inside.
require File.expand_path('../../../../../config/environment', __FILE__)
else
raise "Could not find a config/environment.rb file to require. Please specify this in #{File.expand_path(__FILE__)}"
Copy link
Member Author

Choose a reason for hiding this comment

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

I removed this because rails_admin trips on this but I would love to hear a better workaround.

Copy link
Member

Choose a reason for hiding this comment

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

Change to a puts

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@travisbot
Copy link

This pull request passes (merged a0581c2 into 4f1c75f).

@parndt
Copy link
Member

parndt commented Jun 13, 2012

Great stuff

@travisbot
Copy link

This pull request passes (merged 0a8a2a5 into 4f1c75f).

parndt added a commit that referenced this pull request Jun 20, 2012
Rename all .rb templates which contain erb to .rb.erb.
@parndt parndt merged commit f3366ac into master Jun 20, 2012
ugisozols added a commit that referenced this pull request Jun 22, 2012
ugisozols added a commit that referenced this pull request Jun 22, 2012
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.

3 participants