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

Replace "should" with "is expected to" in Generated Descriptions #2572

Merged
merged 1 commit into from
Oct 31, 2018
Merged

Replace "should" with "is expected to" in Generated Descriptions #2572

merged 1 commit into from
Oct 31, 2018

Conversation

johnsyweb
Copy link
Contributor

@JonRowe
Copy link
Member

JonRowe commented Oct 13, 2018

You'll need to pin this to your expectations PR to pass, which is our normal practise, then we remove the pin when merged.

Copy link
Member

@benoittgt benoittgt left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the PR

@johnsyweb
Copy link
Contributor Author

History re-written as requested.

@JonRowe JonRowe merged commit e5e4ad7 into rspec:master Oct 31, 2018
@johnsyweb johnsyweb deleted the paj/replace-should-with-is-expected-to branch October 31, 2018 22:38
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Nov 2, 2019
Update ruby-rspec-expectations to 3.9.0.

pkgsr change: Add "USE_LANGUAGES=	# none".


### 3.9.0 / 2019-10-02
[Full Changelog](http://github.com/rspec/rspec-expectations/compare/v3.8.5...v3.9.0)

Enhancements:

* The `respond_to` matcher now uses the signature from `initialize` to validate checks
  for `new` (unless `new` is non standard). (Jon Rowe, #1072)
* Generated descriptions for matchers now use `is expected to` rather than `should` in
  line with our preferred DSL. (Pete Johns, #1080, rspec/rspec-core#2572)
* Add the ability to re-raise expectation errors when matching
  with `match_when_negated` blocks. (Jon Rowe, #1130)
* Add a warning when an empty diff is produce due to identical inspect output.
  (Benoit Tigeot, #1126)

### 3.8.6 / 2019-10-07

Bug Fixes:

* Revert #1125 due to the change being incompatible with our semantic versioning
  policy.

### 3.8.5 / 2019-10-02
[Full Changelog](http://github.com/rspec/rspec-expectations/compare/v3.8.4...v3.8.5)

Bug Fixes:

* Prevent unsupported implicit block expectation syntax from being used.
  (Phil Pirozhkov, #1125)

### 3.8.4 / 2019-06-10
[Full Changelog](http://github.com/rspec/rspec-expectations/compare/v3.8.3...v3.8.4)

Bug Fixes:

* Prevent false negatives when checking objects for the methods required to run the
  the `be_an_instance_of` and `be_kind_of` matchers. (Nazar Matus, #1112)
MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
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.

4 participants