-
Notifications
You must be signed in to change notification settings - Fork 731
Remove obsolete OrEqualTo methods
#2269
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 obsolete OrEqualTo methods
#2269
Conversation
OrEqualTo methods
Pull Request Test Coverage Report for Build 5992739567
💛 - Coveralls |
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
4cb2070 to
b6f1650
Compare
OrEqualTo methodsOrEqualTo methods
jnyrup
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍
Here's a few more places that needs to be changed as well.
docs/_data/mstest-migration/assert.yml:249: actual.Should().BeLessOrEqualTo(expected);
docs/_data/mstest-migration/assert.yml:261: actual.Should().BeGreaterOrEqualTo(expected);
docs/_data/mstest-migration/assert.yml:297: actual.Should().BeGreaterOrEqualTo(expected);
docs/_data/mstest-migration/assert.yml:309: actual.Should().BeLessOrEqualTo(expected);
docs/_data/tips/collections.yml:123: actual.Count().Should().BeGreaterOrEqualTo(k);
docs/_data/tips/collections.yml:126: actual.Should().HaveCountGreaterOrEqualTo(k);
docs/_data/tips/collections.yml:147: actual.Count().Should().BeLessOrEqualTo(k);
docs/_data/tips/collections.yml:150: actual.Should().HaveCountLessOrEqualTo(k);
Src/FluentAssertions/Numeric/NumericAssertions.cs:230: public AndConstraint<TAssertions> BeLessOrEqualTo
Src/FluentAssertions/Numeric/NumericAssertions.cs:292: public AndConstraint<TAssertions> BeGreaterOrEqualTo
You can always count on @jnyrup to be thorough 😜💪 |
e59c3bc to
34281cc
Compare
34281cc to
d62eaa2
Compare
Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com>
d62eaa2 to
0925d9f
Compare
|
Can anybody get this Qodana issues? 😂 This files don't exist anymore... Ah.. nevermind.. They are gone now :) |
* Remove obsolete `OrEqualTo`: `GenericCollectionAssertions` * Remove obsolete `OrEqualTo`: `ComparableTypeAssertions` * Remove obsolete `OrEqualTo`: `SimpleTimeSpanAssertions` * Remove obsolete `OrEqualTo`: `ExecutionTimeAssertions` * Remove obsolete `OrEqualTo`: `NumericAssertions` * Accept API changes * Update releases.md Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com> * Fix migration docs --------- Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com>
This refs: #1677
IMPORTANT
./build.sh --target spellcheckor.\build.ps1 --target spellcheckbefore pushing and check the good outcome