Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[workflows] Remove sync-release-repo.yml #73682

Merged
merged 2 commits into from
Nov 29, 2023

Conversation

tstellar
Copy link
Collaborator

@tstellar tstellar commented Nov 28, 2023

This should not be needed now that the project uses PRs, plus the OpenSSF Scorecard report flagged this as insecure.

This should not be needed now that the project uses PRs and the
OpenSSF Scorecard report as flagging this as insecure.
@llvmbot
Copy link
Collaborator

llvmbot commented Nov 28, 2023

@llvm/pr-subscribers-github-workflow

Author: Tom Stellard (tstellar)

Changes

This should not be needed now that the project uses PRs and the OpenSSF Scorecard report as flagging this as insecure.


Full diff: https://github.com/llvm/llvm-project/pull/73682.diff

1 Files Affected:

  • (removed) .github/workflows/sync-release-repo.yml (-17)
diff --git a/.github/workflows/sync-release-repo.yml b/.github/workflows/sync-release-repo.yml
deleted file mode 100644
index c328a9133e179de..000000000000000
--- a/.github/workflows/sync-release-repo.yml
+++ /dev/null
@@ -1,17 +0,0 @@
-name: Sync Release Repositories
-permissions:
-  contents: write
-on:
-  workflow_dispatch
-env:
-  RELEASE_BRANCH: release/17.x
-  GH_TOKEN: ${{ secrets.RELEASE_WORKFLOW_PUSH_SECRET }}
-jobs:
-  sync:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Fetch LLVM sources
-        uses: actions/checkout@v4
-      - name: Run Sync Script
-        run: |
-          llvm/utils/git/sync-release-repo.sh

Copy link
Collaborator

@tru tru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah let's axe it.

@tru
Copy link
Collaborator

tru commented Nov 28, 2023

The sh script can be removed as well.

@tstellar tstellar merged commit dc16964 into llvm:main Nov 29, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants