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

Validate ODL Version in csproj against builder.versions.settings.targets and update ODataLibPackageDependency to 8.2.3 #1389

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

WanjohiSammy
Copy link
Contributor

@WanjohiSammy WanjohiSammy commented Jan 12, 2025

Issues

This pull request fixes #1387.

Description:

This change involves:

  • Add PowerShell script to validate the OData Package version in Microsoft.AspNetCore.OData.csproj against the version in builder.versions.settings.targets.
  • Add a step in the pipeline to validate the OData Package version by running a PowerShell script.
  • Update ODataLibPackageDependency version to [8.2.3, 9.0.0)

This approach is to ensure that the OData Package version in the Microsoft.AspNetCore.OData.csproj file matches the version in builder.versions.settings.targets as we consider the option of moving away hardcoded .nuspec file:

<dependencies>
<group targetFramework=".net8.0">
<dependency id="Microsoft.OData.ModelBuilder" version="$ODataModelBuilderPackageDependency$" />
<dependency id="Microsoft.OData.Core" version="$ODataLibPackageDependency$" />
<dependency id="Microsoft.OData.Edm" version="$ODataLibPackageDependency$" />
<dependency id="Microsoft.Spatial" version="$ODataLibPackageDependency$" />
</group>
</dependencies>

@WanjohiSammy WanjohiSammy changed the title Add a step in pipeline to validate OData Package Versions Validate DL Package Version in csproj against builder.versions.settings.targets and update ODataLibPackageDependency to 8.2.3 Jan 12, 2025
@WanjohiSammy WanjohiSammy changed the title Validate DL Package Version in csproj against builder.versions.settings.targets and update ODataLibPackageDependency to 8.2.3 Validate ODL Version in csproj against builder.versions.settings.targets and update ODataLibPackageDependency to 8.2.3 Jan 12, 2025
@WanjohiSammy WanjohiSammy requested review from Copilot, habbes, marabooy and xuzhg and removed request for Copilot January 12, 2025 10:35

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • tool/ValidateODataPackageVersions.ps1: Language not supported
  • tool/builder.versions.settings.targets: Language not supported
xuzhg
xuzhg previously approved these changes Jan 14, 2025

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (2)
  • tool/ValidateODataPackageVersions.ps1: Language not supported
  • tool/builder.versions.settings.targets: Language not supported
@WanjohiSammy WanjohiSammy merged commit 275985b into main Jan 15, 2025
4 checks passed
@WanjohiSammy WanjohiSammy deleted the fix/1387-fix-incorrect-odatalib-dependency-version branch January 15, 2025 09:26
WanjohiSammy added a commit that referenced this pull request Jan 30, 2025
…ets and update ODataLibPackageDependency to 8.2.3 (#1389)

* Add a step to pipeline to validate Validate OData Package Versions

* Update ODataLibPackageDependency version to [8.2.3, 9.0.0)

* Bump version to 9.1.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Version mismatch in nuget package 9.1.2
2 participants