Update README to include caveat of auto require

NoRedInk / rspec-retry

retry randomly failing rspec example

MIT License

584 stars 94 forks source link

Update README to include caveat of auto require #87

Closed sb8244 closed 6 years ago

sb8244 commented 6 years ago

I ran into this bug where I saw the RSpec::Retry code wrapping our examples in our non-feature test suite. While it doesn't execute except for JS tests, it was a confusing and unexpected behavior. I wanted to document it for future developers.

This is a really old PR that I had botched (see https://github.com/sb8244/rspec-retry/pull/1), so I'm finally moving it to this repo.

michaelglass commented 6 years ago

I'm sorry, I'm going to reject this PR as it makes the documentation less pithy and documents the behavior of ruby & rspec rather than the behavior of rspec-retry. I'm not sure where this type of "user guides" stuff belongs but I'm vetoing the readme.

sb8244 commented 6 years ago

No worries. Cheers