Skip to content

Commit 8245d6f

Browse files
ANcpLuaCopilot
andcommitted
fix: restore BannedSymbols filenames and remove unsafe publish workflow
BannedApiAnalyzers requires files named BannedSymbols.txt / BannedSymbols.*.txt. Renaming to Default.txt broke RS0030 detection. - Rename Default.txt → BannedSymbols.txt - Rename NewtonsoftJson.txt → BannedSymbols.NewtonsoftJson.txt - Remove nuget-publish.yml (tag-triggered publish bypassed CI tests) - ci.yml deploy job already handles publishing with proper test gating Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent edbdfe3 commit 8245d6f

7 files changed

Lines changed: 8 additions & 63 deletions

File tree

.github/workflows/nuget-publish.yml

Lines changed: 0 additions & 55 deletions
This file was deleted.

src/ANcpLua.NET.Sdk.Test.nuspec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@
5454
<file src="Config/Style/GeneratedFiles.editorconfig" target="Config/Style/"/>
5555

5656
<!-- Config/BannedSymbols - Banned API symbol lists -->
57-
<file src="Config/BannedSymbols/Default.txt" target="Config/BannedSymbols/"/>
58-
<file src="Config/BannedSymbols/NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
57+
<file src="Config/BannedSymbols/BannedSymbols.txt" target="Config/BannedSymbols/"/>
58+
<file src="Config/BannedSymbols/BannedSymbols.NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
5959

6060
<!-- Config root -->
6161
<file src="Config/default.runsettings" target="Config/"/>

src/ANcpLua.NET.Sdk.Web.nuspec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@
5454
<file src="Config/Style/GeneratedFiles.editorconfig" target="Config/Style/"/>
5555

5656
<!-- Config/BannedSymbols - Banned API symbol lists -->
57-
<file src="Config/BannedSymbols/Default.txt" target="Config/BannedSymbols/"/>
58-
<file src="Config/BannedSymbols/NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
57+
<file src="Config/BannedSymbols/BannedSymbols.txt" target="Config/BannedSymbols/"/>
58+
<file src="Config/BannedSymbols/BannedSymbols.NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
5959

6060
<!-- Config root -->
6161
<file src="Config/default.runsettings" target="Config/"/>

src/ANcpLua.NET.Sdk.nuspec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@
5454
<file src="Config/Style/GeneratedFiles.editorconfig" target="Config/Style/"/>
5555

5656
<!-- Config/BannedSymbols - Banned API symbol lists -->
57-
<file src="Config/BannedSymbols/Default.txt" target="Config/BannedSymbols/"/>
58-
<file src="Config/BannedSymbols/NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
57+
<file src="Config/BannedSymbols/BannedSymbols.txt" target="Config/BannedSymbols/"/>
58+
<file src="Config/BannedSymbols/BannedSymbols.NewtonsoftJson.txt" target="Config/BannedSymbols/"/>
5959

6060
<!-- Config root -->
6161
<file src="Config/default.runsettings" target="Config/"/>

src/Build/Common/Common.targets

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,8 +231,8 @@
231231
<!-- SDK-injected analyzer packages are now handled via GlobalPackages.props -->
232232

233233
<ItemGroup>
234-
<AdditionalFiles Include="$(MSBuildThisFileDirectory)../../Config/BannedSymbols/Default.txt" Condition="'$(IncludeDefaultBannedSymbols)' != 'false'" InProject="false"/>
235-
<AdditionalFiles Include="$(MSBuildThisFileDirectory)../../Config/BannedSymbols/NewtonsoftJson.txt" Condition="'$(BannedNewtonsoftJsonSymbols)' != 'false'" InProject="false"/>
234+
<AdditionalFiles Include="$(MSBuildThisFileDirectory)../../Config/BannedSymbols/BannedSymbols.txt" Condition="'$(IncludeDefaultBannedSymbols)' != 'false'" InProject="false"/>
235+
<AdditionalFiles Include="$(MSBuildThisFileDirectory)../../Config/BannedSymbols/BannedSymbols.NewtonsoftJson.txt" Condition="'$(BannedNewtonsoftJsonSymbols)' != 'false'" InProject="false"/>
236236
</ItemGroup>
237237

238238
<ItemGroup Condition="'$(Language)' == 'C#' AND ('$(ImplicitUsings)' == 'true' or '$(ImplicitUsings)' == 'enable')">
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)