From 488d60708fdb7971315827fb5960f9fd0e6102a5 Mon Sep 17 00:00:00 2001 From: Austin Drenski Date: Tue, 16 Jan 2024 13:47:46 -0500 Subject: [PATCH] chore: Fix props to supprot more than one project Ran across this while working on open-feature/dotnet-sdk-contrib#127, and tldr; if there's more than one project under `src/`, having the `` hard-coded in `build/Common.prod.props` causes MSBuild to panic. There are two options to fix this: 1. ```diff - OpenFeature + $(MSBuildProjectName) ``` 2. ```diff - OpenFeature ``` Since NuGet defaults `PackageId` to the assembly name, I'm opting for (2), but if I've overlooked some nuance/custom build steps that require setting `PackageId` explicitly, then we could just as well fallback to (1). Signed-off-by: Austin Drenski --- .github/workflows/release.yml | 2 +- build/Common.prod.props | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 644342a7..52c68730 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -42,7 +42,7 @@ jobs: - name: Pack if: ${{ steps.release.outputs.releases_created }} run: | - dotnet pack OpenFeature.proj --configuration Release --no-build -p:PackageID=OpenFeature + dotnet pack OpenFeature.proj --configuration Release --no-build - name: Publish to Nuget if: ${{ steps.release.outputs.releases_created }} diff --git a/build/Common.prod.props b/build/Common.prod.props index 49e454c5..63a52d44 100644 --- a/build/Common.prod.props +++ b/build/Common.prod.props @@ -11,7 +11,6 @@ https://github.com/open-feature/dotnet-sdk OpenFeature is an open standard for feature flag management, created to support a robust feature flag ecosystem using cloud native technologies. OpenFeature will provide a unified API and SDK, and a developer-first, cloud-native implementation, with extensibility for open source and commercial offerings. Feature;OpenFeature;Flags; - OpenFeature openfeature-icon.png https://openfeature.dev Apache-2.0