Skip to content

Remove Feedz publishing from CI - #1380

Merged
meziantou merged 1 commit into
mainfrom
feature/remove-feedz-publishing-1683d9
Sep 6, 2026
Merged

meziantou merged 1 commit into
mainfrom
feature/remove-feedz-publishing-1683d9

Conversation

@meziantou

Copy link
Copy Markdown
Owner

What changed

Removes the Feedz feed from the deploy job of .github/workflows/ci.yml:

  • The Publish NuGet packages step no longer branches on GITHUB_REF. It always pushes to nuget.org, and the whole step is gated on github.ref == 'refs/heads/main' — previously the else branch (non-main) is what pushed to https://f.feedz.io/meziantou/meziantou-analyzer/nuget/index.json.
  • Dropped the FeedzApiKey env var and its secrets.FEEDZ_APIKEY usage.
  • Gated the NuGet login (OIDC → temp API key) step on main as well, since its only consumer is the push step. It was previously minting a temporary nuget.org API key on every non-fork PR run with nothing left to use it.

Why

Feedz publishing of CI builds is no longer wanted.

Notes for reviewers

  • The deploy job itself still runs on PRs (it checks out, downloads the nuget artifact and sets up .NET), so any required status check on deploy keeps reporting instead of being skipped. Both remaining publish-related steps simply no-op off main.
  • No source or test code changed, so there was nothing to build or run; the workflow file parses as valid YAML and no feedz reference remains in the repo.
  • The FEEDZ_APIKEY repository secret is now unused and can be deleted in the GitHub settings.

Packages were pushed to the Feedz feed for every non-main branch. Drop
that branch of the publish step so CI only pushes to nuget.org, and gate
the push (and the OIDC NuGet login it depends on) on the main branch.
@meziantou
meziantou merged commit 5c6da9e into main Sep 6, 2026
13 checks passed
@meziantou
meziantou deleted the feature/remove-feedz-publishing-1683d9 branch September 6, 2026 00:03
This was referenced Sep 6, 2026
This was referenced Sep 25, 2026
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.

1 participant