Skip to content

Commit 5a093a1

Browse files
Fix CS7035 warning in dependabot jobs (#2849)
Fix `CS7035` warnings from `$GITHUB_RUN_NUMBER` not being set when dependabot jobs run.
1 parent e73ab5f commit 5a093a1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

eng/Common.targets

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@
2525

2626
<Target Name="CustomizeVersions" AfterTargets="MinVer" Condition="'$(GITHUB_ACTIONS)' == 'true'">
2727
<PropertyGroup>
28-
<FileVersion>$([MSBuild]::ValueOrDefault('$(MinVerMajor)', '8')).$([MSBuild]::ValueOrDefault('$(MinVerMinor)', '6')).$([MSBuild]::ValueOrDefault('$(MinVerPatch)', '0')).$(GITHUB_RUN_NUMBER)</FileVersion>
28+
<FileVersion>$([MSBuild]::ValueOrDefault('$(MinVerMajor)', '8')).$([MSBuild]::ValueOrDefault('$(MinVerMinor)', '6')).$([MSBuild]::ValueOrDefault('$(MinVerPatch)', '0')).$([MSBuild]::ValueOrDefault('$(GITHUB_RUN_NUMBER)', '0'))</FileVersion>
2929
</PropertyGroup>
3030
<PropertyGroup Condition="$(GITHUB_REF.StartsWith(`refs/pull/`))">
31-
<PackageVersion>$(MinVerMajor).$(MinVerMinor).$(MinVerPatch)-pr.$(GITHUB_REF_NAME.Replace(`/merge`, ``)).$(GITHUB_RUN_NUMBER)</PackageVersion>
31+
<PackageVersion>$(MinVerMajor).$(MinVerMinor).$(MinVerPatch)-pr.$(GITHUB_REF_NAME.Replace(`/merge`, ``)).$([MSBuild]::ValueOrDefault('$(GITHUB_RUN_NUMBER)', '0'))</PackageVersion>
3232
</PropertyGroup>
3333
</Target>
3434

0 commit comments

Comments
 (0)