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

allow multiple forms for $.job.commit-message-options.include-scope property #11778

Merged
merged 1 commit into from
Mar 10, 2025

Conversation

brettfo
Copy link
Contributor

@brettfo brettfo commented Mar 10, 2025

To help with dependabot/cli#408 this PR allows the following values for the include-scope property:

  • true bool
  • false
  • "true" stringified bool
  • "false"
  • null null - false

The old values are still supported to allow for older CLI versions to still work.

@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Mar 10, 2025
@brettfo brettfo marked this pull request as ready for review March 10, 2025 18:16
@brettfo brettfo requested a review from a team as a code owner March 10, 2025 18:16
@rhyskoedijk
Copy link

thanks @brettfo

@randhircs randhircs force-pushed the dev/brettfo/nuget-job-deserialization branch from 5986753 to dfa3d43 Compare March 10, 2025 19:18
@randhircs randhircs merged commit 511f4c7 into main Mar 10, 2025
77 checks passed
@randhircs randhircs deleted the dev/brettfo/nuget-job-deserialization branch March 10, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants