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 a deprecation warning to 2.99 for accessing let/subject from after(:all)
#1081
Add a deprecation warning to 2.99 for accessing let/subject from after(:all)
#1081
Conversation
… in after(:all)
Want to review this @myronmarston ? |
|
||
This is deprecated behavior that will not be supported in RSpec 3. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you move this line down below the main blurb?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes more sense to me to have the statement after the explanation, if it's an issue I can move it back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fine, was just curious.
Looks good...left a few comments. |
[skip ci]
Merge when green. |
I skipped the CI on the changlog change so it's green already. |
…efore_all Add a deprecation warning to 2.99 for accessing let/subject from `after(:all)`
…n_warning_let_subject_before_all Add a deprecation warning to 2.99 for accessing let/subject from `after(:all)` --- This commit was imported from rspec/rspec-core@a807624.
Follow up to #1078