Make sure developers don't accidentally update to v8 without understanding the new license by dennisdoomen · Pull Request #2988 · fluentassertions/fluentassertions · GitHub
Skip to content

Make sure developers don't accidentally update to v8 without understanding the new license#2988

Merged
dennisdoomen merged 1 commit into
fluentassertions:mainfrom
dennisdoomen:prevent-accidental-update
Jan 23, 2025
Merged

Make sure developers don't accidentally update to v8 without understanding the new license#2988
dennisdoomen merged 1 commit into
fluentassertions:mainfrom
dennisdoomen:prevent-accidental-update

Conversation

@dennisdoomen

Copy link
Copy Markdown
Member

Added the PackageRequireLicenseAcceptance to the package to request people to accept the license so they don't accidentally update to v8 without making sure they comply to the new license.

Note Only works on Visual Studio. Rider doesn't support this.

@dennisdoomen dennisdoomen requested a review from jnyrup January 23, 2025 07:07
@github-actions

Copy link
Copy Markdown

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 12924039942

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.404%

Totals Coverage Status
Change from base Build 12892609508: 0.0%
Covered Lines: 12290
Relevant Lines: 12474

💛 - Coveralls

@dennisdoomen dennisdoomen added the building Building and Infrastructure of Fluent Assertions label Jan 23, 2025
@dennisdoomen dennisdoomen added this to the 8.1.0 milestone Jan 23, 2025
@dennisdoomen dennisdoomen merged commit 4760f3c into fluentassertions:main Jan 23, 2025
@dennisdoomen dennisdoomen deleted the prevent-accidental-update branch January 23, 2025 09:15
This was referenced Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

building Building and Infrastructure of Fluent Assertions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants