Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Update rake task docs with note about LoadError#1642

Merged
JonRowe merged 1 commit into
masterfrom
update_rake_task_docs
Jul 16, 2014
Merged

Update rake task docs with note about LoadError#1642
JonRowe merged 1 commit into
masterfrom
update_rake_task_docs

Conversation

@JonRowe
Copy link
Copy Markdown
Member

@JonRowe JonRowe commented Jul 16, 2014

Update rake task docs with note(s) about using rescue LoadError to allow usage in production.

Fixes #1638

Comment thread features/command_line/rake_task.feature Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this should just be "We recommend....". I'm not sure what value "By default" adds here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated.

@myronmarston
Copy link
Copy Markdown
Member

LGTM, merge when green.

JonRowe added a commit that referenced this pull request Jul 16, 2014
Update rake task docs with note about LoadError
@JonRowe JonRowe merged commit 6635043 into master Jul 16, 2014
@JonRowe JonRowe deleted the update_rake_task_docs branch July 16, 2014 23:30
JonRowe added a commit that referenced this pull request Jul 16, 2014
Update rake task docs with note about LoadError
@JonRowe
Copy link
Copy Markdown
Member Author

JonRowe commented Jul 16, 2014

Cherry picked to 3-0-maintenance for completeness.

@afeld
Copy link
Copy Markdown

afeld commented Jul 17, 2014

👍

MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
Update rake task docs with note about LoadError
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
…e_task_docs

Update rake task docs with note about LoadError

---
This commit was imported from rspec/rspec-core@1d051f2.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rake task recommendation causes issues in production

3 participants