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

Tweak deprecation message for symbol descriptions.#1667

Merged
JonRowe merged 2 commits into
2-99-maintenancefrom
tweak_inner_symbol_deprecation
Aug 19, 2014
Merged

Tweak deprecation message for symbol descriptions.#1667
JonRowe merged 2 commits into
2-99-maintenancefrom
tweak_inner_symbol_deprecation

Conversation

@JonRowe
Copy link
Copy Markdown
Member

@JonRowe JonRowe commented Aug 19, 2014

Tweak deprecation message for symbol descriptions to indicate it will effect described_class, this should fix #1666.

Note I added a spec covering this.

Comment thread spec/rspec/core/example_group_spec.rb 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.

This line should be removed.

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.

Vim buffer error ;) the amount of time I removed this...

@myronmarston
Copy link
Copy Markdown
Member

LGTM besides my comments.

@JonRowe
Copy link
Copy Markdown
Member Author

JonRowe commented Aug 19, 2014

I'm going to merge when green, and not going to add a changelog (as it's just tweaking an existing deprecation).

@myronmarston
Copy link
Copy Markdown
Member

not going to add a changelog (as it's just tweaking an existing deprecation).

Normally I would agree with this but I plan to cut a 2.99.2 release with this later today (since we have no other planned 2.99 changes, no reason to wait...) and there are no other changes in 2-99-maintenance...so I think it's worth putting a changelog entry about improving the deprecation message. Otherwise the changelog will suggest that 2.99.2 includes no changes.

@JonRowe
Copy link
Copy Markdown
Member Author

JonRowe commented Aug 19, 2014

Ok, I'll add one then.

JonRowe added a commit that referenced this pull request Aug 19, 2014
Tweak deprecation message for symbol descriptions.
@JonRowe JonRowe merged commit 00eceeb into 2-99-maintenance Aug 19, 2014
@JonRowe JonRowe deleted the tweak_inner_symbol_deprecation branch August 19, 2014 17:42
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
[skip ci]

---
This commit was imported from rspec/rspec-core@0f6ce5b.
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
…r_symbol_deprecation

Tweak deprecation message for symbol descriptions.

---
This commit was imported from rspec/rspec-core@00eceeb.
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.

2 participants