Skip to content

Commit

Permalink
[0.74] Temporarily disable code-signing of NuGet packages (#14031)
Browse files Browse the repository at this point in the history
## Description

This PR temporarily disables NuGet code-signing during publish and also forces all new projects to include our public ADO feed which will contain unsigned packages.

### Type of Change
- Bug fix (non-breaking change which fixes an issue)

### Why
See #14030 for why this is necessary.

### What
See above.

## Screenshots
N/A

## Testing
N/A

## Changelog
Should this change be included in the release notes: _yes_

[0.74] Temporarily disable code-signing of NuGet packages
  • Loading branch information
jonthysell authored Oct 24, 2024
1 parent c4e39ef commit d524e2a
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 7 deletions.
8 changes: 4 additions & 4 deletions .ado/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ extends:
packMicrosoftReactNativeManaged: true
packMicrosoftReactNativeManagedCodeGen: true
${{ if or(eq(variables['EnableCodesign'], 'true'), endsWith(variables['Build.SourceBranchName'], '-stable')) }}: # Sign if EnableCodeSign or on *-stable release builds
signMicrosoft: true
signMicrosoft: false # Temporarily disabled for all builds, see issue #14030
slices:
- platform: x64
configuration: Release
Expand All @@ -485,7 +485,7 @@ extends:
# packMicrosoftReactNativeManaged: true
# packMicrosoftReactNativeManagedCodeGen: true
${{ if or(eq(variables['EnableCodesign'], 'true'), endsWith(variables['Build.SourceBranchName'], '-stable')) }}: # Sign if EnableCodeSign or on *-stable release builds
signMicrosoft: true
signMicrosoft: false # Temporarily disabled for all builds, see issue #14030
slices:
- platform: x64
configuration: Release
Expand All @@ -508,7 +508,7 @@ extends:
nugetroot: $(System.DefaultWorkingDirectory)\Desktop
packDesktop: true
${{ if or(eq(variables['EnableCodesign'], 'true'), endsWith(variables['Build.SourceBranchName'], '-stable')) }}: # Sign if EnableCodeSign or on *-stable release builds
signMicrosoft: true
signMicrosoft: false # Temporarily disabled for all builds, see issue #14030
slices:
- platform: x64
configuration: Release
Expand All @@ -531,7 +531,7 @@ extends:
nugetroot: $(System.DefaultWorkingDirectory)\Desktop
packDesktop: true
${{ if or(eq(variables['EnableCodesign'], 'true'), endsWith(variables['Build.SourceBranchName'], '-stable')) }}: # Sign if EnableCodeSign or on *-stable release builds
signMicrosoft: true
signMicrosoft: false # Temporarily disabled for all builds, see issue #14030
slices:
- platform: x64
configuration: Release
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "[0.74] Temporarily disable code-signing of NuGet packages",
"packageName": "@react-native-windows/cli",
"email": "[email protected]",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "[0.74] Temporarily disable code-signing of NuGet packages",
"packageName": "react-native-windows",
"email": "[email protected]",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,8 @@ export async function copyProjectTemplateAndReplace(

useExperimentalNuget: options.experimentalNuGetDependency,
nuGetTestFeed: options.nuGetTestFeed,
nuGetADOFeed: nugetVersion.startsWith('0.0.0-'),
// eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
nuGetADOFeed: true || nugetVersion.startsWith('0.0.0-'), // Temporary true for all new projects until code-signing is restored, see issue #14030

// cpp template variables
useWinUI3: options.useWinUI3,
Expand Down
2 changes: 1 addition & 1 deletion vnext/templates/cpp-app/template.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ async function getFileMappings(config = {}, options = {}) {
devMode,

useNuGets: !devMode, // default is to use published NuGets except in devMode, change to true here if you want to test devMode and nugets simultaneously
addReactNativePublicAdoFeed: isCanary,
addReactNativePublicAdoFeed: true || isCanary, // Temporary true for all new projects until code-signing is restored, see issue #14030

cppNugetPackages,
};
Expand Down
2 changes: 1 addition & 1 deletion vnext/templates/cpp-lib/template.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ async function getFileMappings(config = {}, options = {}) {
devMode,

useNuGets: !devMode, // default is to use published NuGets except in devMode, change to true here if you want to test devMode and nugets simultaneously
addReactNativePublicAdoFeed: isCanary,
addReactNativePublicAdoFeed: true || isCanary, // Temporary true for all new projects until code-signing is restored, see issue #14030

cppNugetPackages,
};
Expand Down

0 comments on commit d524e2a

Please sign in to comment.