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

Bump FluentAssertions from 7.0.0 to 8.0.0 #3934

Closed
wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 14, 2025

Bumps FluentAssertions from 7.0.0 to 8.0.0.

Release notes

Sourced from FluentAssertions's releases.

8.0.0

What's Changed

License change

Breaking Changes

New features

Improvements

... (truncated)

Commits
  • 901c8fa Merge pull request #2947 from fluentassertions/release-8.0
  • df7e9bf Update ownership and license
  • 65d78e2 Merge pull request #2945 from fluentassertions/main
  • bfbf509 Update landing page (#2944)
  • f77fe32 Bump the xunit group with 3 updates
  • 266fede Bump Microsoft.Testing.Extensions.TrxReport from 1.5.0 to 1.5.1
  • e5283d0 Bump Meziantou.Analyzer from 2.0.184 to 2.0.186
  • 3d8f03f Fix several link texts
  • 7426493 Fix release note PR-link
  • bcc974b Simplify FirstOrDefault() + null check to Any()
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from a team as a code owner January 14, 2025 12:22
@dependabot dependabot bot added .NET Pull requests that update .net code dependencies labels Jan 14, 2025
Copy link
Contributor

@Kielek Kielek left a comment

Choose a reason for hiding this comment

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

Blocking merge. We need to double check if we can use this package.
It changed the license from Apache 2.0 to some custom version: fluentassertions/fluentassertions#2943

@nicolas17
Copy link

I'm surprised dependabot blindly updates libraries even if the license changed and without even warning about it.

@Kielek
Copy link
Contributor

Kielek commented Jan 14, 2025

@nicolas17, it references release notes. It is first line there.

@nrcventura
Copy link
Member

SIG meeting update: We discussed whether we should just keep this reference pinned to 7.0.0 or to migrate away from this library. To keep things simple (especially for any forks of this project), and following the precedent in other repos, we decided that we should just migrate to the xunit assertion library.

@mikes-gh
Copy link

@Kielek whats your migration stratergy? Regex find and replace?

@Kielek
Copy link
Contributor

Kielek commented Jan 16, 2025

@mikes-gh, you have couple of options here:

  1. Full/semi manual. VS is pretty smart to suggest correct fixes, especially if changes follow similar pattern.
  2. If your project does not have a lot of changes, you can try to make most common by the Regexes, for leftovers go to first option.
  3. If you have thousands of assertions, you could consider creating auto code fixes for most common scenarios. Ref https://learn.microsoft.com/en-us/dotnet/csharp/roslyn-sdk/tutorials/how-to-write-csharp-analyzer-code-fix

For this project, the fastest option was go throw mix of 1 (mostly) and 2.

@mikes-gh
Copy link

Thanks @Kielek we have 6800 Should().Be( and probably more assertions than that. I guess there's no magic bullet. Hopefully someone will publish some code fixes. They would be different for NUnit vs XUnit (which I think is more popular). We may go Shoudly which might be an easier transition.

@dependabot dependabot bot force-pushed the dependabot/nuget/FluentAssertions-8.0.0 branch from 20f3f44 to 80709d5 Compare January 17, 2025 06:40
Bumps [FluentAssertions](https://github.com/fluentassertions/fluentassertions) from 7.0.0 to 8.0.0.
- [Release notes](https://github.com/fluentassertions/fluentassertions/releases)
- [Changelog](https://github.com/fluentassertions/fluentassertions/blob/develop/AcceptApiChanges.ps1)
- [Commits](fluentassertions/fluentassertions@7.0.0...8.0.0)

---
updated-dependencies:
- dependency-name: FluentAssertions
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/nuget/FluentAssertions-8.0.0 branch from 80709d5 to 0b36f7d Compare January 17, 2025 06:46
Copy link
Contributor Author

dependabot bot commented on behalf of github Jan 17, 2025

Looks like FluentAssertions is no longer a dependency, so this is no longer needed.

@dependabot dependabot bot closed this Jan 17, 2025
@dependabot dependabot bot deleted the dependabot/nuget/FluentAssertions-8.0.0 branch January 17, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies .NET Pull requests that update .net code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants