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

Remove deprecated :example_group example metadata sub-hash #2851

Merged
merged 1 commit into from
Feb 10, 2021

Conversation

pirj
Copy link
Member

@pirj pirj commented Jan 12, 2021

@pirj pirj self-assigned this Jan 12, 2021
@pirj
Copy link
Member Author

pirj commented Jan 12, 2021

@JonRowe I would really appreciate your help here. Specs are failing and I guess I did something wrong.

@pirj pirj added this to the 4.0 milestone Jan 28, 2021
@pirj

This comment has been minimized.

@pirj pirj force-pushed the remove-example_group-subhash branch from ac3ae2d to e1dfc63 Compare February 10, 2021 07:55
@pirj pirj changed the title WIP Remove example_group metadata sub-hash Remove example_group metadata sub-hash Feb 10, 2021
@pirj pirj changed the title Remove example_group metadata sub-hash Remove :example_group example metadata sub-hash Feb 10, 2021
@pirj pirj changed the title Remove :example_group example metadata sub-hash Remove deprecated :example_group example metadata sub-hash Feb 10, 2021
@pirj pirj marked this pull request as ready for review February 10, 2021 08:05
@pirj pirj mentioned this pull request Feb 10, 2021
53 tasks
@pirj pirj force-pushed the remove-example_group-subhash branch from e1dfc63 to d7603e5 Compare February 10, 2021 08:14
lib/rspec/core/metadata.rb Outdated Show resolved Hide resolved
lib/rspec/core/metadata.rb Outdated Show resolved Hide resolved
Changelog.md Outdated
@@ -13,6 +13,8 @@ Breaking Changes:
* Raise on usage of metadata on suite-level scopes. (Phil Pirozhkov, #2849)
* Raise an error when `fail_fast` is configured with
an unsupported value. (Phil Pirozhkov, #2849)
* Remove deprecated exposition of example group's metadata
in example's metadata `:example_group` subhash. (Phil Pirozhkov, #2851)
Copy link
Member

Choose a reason for hiding this comment

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

This needs rewording but I havent had enough coffee yet to process how.

@JonRowe
Copy link
Member

JonRowe commented Feb 10, 2021

This looks good to me overall

@pirj pirj force-pushed the remove-example_group-subhash branch from 18e12b3 to 762934a Compare February 10, 2021 13:30
Changelog.md Outdated Show resolved Hide resolved
@pirj pirj force-pushed the remove-example_group-subhash branch from eb5c579 to 9daaf1f Compare February 10, 2021 21:49
@pirj pirj merged commit 0ea90bb into 4-0-dev Feb 10, 2021
@pirj pirj deleted the remove-example_group-subhash branch February 10, 2021 22:15
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 19, 2021
…mple_group-subhash

Remove deprecated `:example_group` example metadata sub-hash

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

2 participants