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

Document the syntax for providing a skip reason in metadata #2770

Merged
merged 1 commit into from Sep 29, 2020

Conversation

jxa
Copy link
Contributor

@jxa jxa commented Sep 29, 2020

Adds a feature to describe the process for skipping a spec via metadata, for example:

I knew this was a thing but when I went looking for the documentation I noticed this scenario was absent.

@JonRowe
Copy link
Member

JonRowe commented Sep 29, 2020

Thanks for taking the time to contribute this! If you can just fix the line ending so it has a \n (but not a \n\n which would be a blank line). If you're using a linux / os x using a terminal editor like vim is the easiest way, you just need to save the file.

@jxa jxa force-pushed the add-feature-skip-with-metadata branch from 41a635d to 3fee6e0 Compare September 29, 2020 20:39
@JonRowe JonRowe merged commit 41f4299 into rspec:main Sep 29, 2020
@JonRowe
Copy link
Member

JonRowe commented Sep 29, 2020

Thanks again.

JonRowe added a commit that referenced this pull request Sep 30, 2020
Document the syntax for providing a skip reason in metadata
@JonRowe
Copy link
Member

JonRowe commented Sep 30, 2020

This has been published to relish

MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
Document the syntax for providing a skip reason in metadata
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
…skip-with-metadata

Document the syntax for providing a skip reason in metadata

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

2 participants