Skip to content

ci: remove unused workflows - #530

Closed
jerome-uniswap wants to merge 1 commit into
mainfrom
ci/remove-unused-workflows
Closed

jerome-uniswap wants to merge 1 commit into
mainfrom
ci/remove-unused-workflows

Conversation

@jerome-uniswap

Copy link
Copy Markdown

Summary

CI hardening sweep across Uniswap's public repos: remove workflows that are no longer used, or that fail on every run, and that still run with elevated CI permissions (an OIDC token and/or write access). Unused privileged workflows add risk without adding value.

Removed:

  • .github/workflows/deploy.yml

Recent run history for each is in the review sheet (internal). If one of these is still needed, git revert this PR or tell me and I'll scope it down instead.

🤖 Generated with Claude Code

Remove workflows that are unused or failing and that run with elevated
CI permissions (OIDC token and/or write access). Part of a CI hardening
sweep across public repos. Restore with git revert if one is still needed.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jerome-uniswap
jerome-uniswap requested a review from a team as a code owner October 30, 2026 16:05
@github-actions

Copy link
Copy Markdown

🤖 Claude PR Metadata Generation

Status: ❌ Error
Job: View workflow run


⚠️ Generation Failed

The automated PR title and description generation encountered an error.

Next Steps:

  1. Check the workflow logs for details
  2. Verify the GitHub Actions configuration
  3. Ensure the Anthropic API key is valid

You can manually set the PR title and description, or re-trigger by pushing a new commit.

@jerome-uniswap

Copy link
Copy Markdown
Author

Closing: deploy.yml is the npm release path for @uniswap/universal-router (2.1.0 was published from it), so it shouldn't be removed. Will tighten its release environment instead.

@jerome-uniswap
jerome-uniswap deleted the ci/remove-unused-workflows branch October 30, 2026 16:24
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