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

Improve rails:template undefined LOCATION variable error message fixed #2198

Merged
merged 1 commit into from
Jul 23, 2011
Merged

Improve rails:template undefined LOCATION variable error message fixed #2198

merged 1 commit into from
Jul 23, 2011

Conversation

Karunakar
Copy link
Contributor

this is fixed

@@ -5,11 +5,11 @@ namespace :rails do
desc "Applies the template supplied by LOCATION=/path/to/template"
task :template do
template = ENV["LOCATION"]
raise "No LOCATION value given. Please set LOCATION=/path/to/template" if template.blank?
Copy link
Member

Choose a reason for hiding this comment

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

LOCATION can also be a URL.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ya. we know . It can be a URL. we are just showing the description.

Copy link
Contributor

Choose a reason for hiding this comment

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

we added the same description which is in the rake task

Copy link
Contributor

Choose a reason for hiding this comment

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

we will update the description in the next commit

@spastorino
Copy link
Contributor

Can you squash both commits into one?, thanks

@raghunadhd
Copy link
Contributor

@spastorino It's done. Let it go in :-)

spastorino added a commit that referenced this pull request Jul 23, 2011
Improve rails:template undefined LOCATION variable error message fixed
@spastorino spastorino merged commit 35c87a1 into rails:master Jul 23, 2011
@raghunadhd
Copy link
Contributor

@spastorino Thanks

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

5 participants