Skip to content

chore(ci): Drop the FRAMER_ADMIN_SECRET declaration. - #696

Merged
xespona merged 1 commit into
mainfrom
chore/drop-framer-admin-secret-declaration
Sep 9, 2026
Merged

chore(ci): Drop the FRAMER_ADMIN_SECRET declaration.#696
xespona merged 1 commit into
mainfrom
chore/drop-framer-admin-secret-declaration

Conversation

@xespona

@xespona xespona commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Context

Releases stopped using FRAMER_ADMIN_SECRET in #691, but the declaration stayed so callers passing it wouldn't fail validation. framer/workshop was the only one, and it stopped in framer/workshop#541, so the declaration can go.

Changes

  • Removed the FRAMER_ADMIN_SECRET secret from the reusable workflow. Nothing in the repo reads it now.

The repo secret itself can be deleted separately, in this repo and in workshop.

framer/workshop was the last caller passing it and stopped in framer/workshop#541.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@xespona
xespona marked this pull request as ready for review September 9, 2026 10:09
Copilot AI lite review requested due to automatic review settings September 9, 2026 10:09
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-09T10:11:13.103348Z 2f3b209 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The removed secret is no longer referenced in the repository and the workflow remains internally consistent after the declaration is dropped.

Pull request overview

This PR removes the now-unused FRAMER_ADMIN_SECRET secret declaration from the reusable “Submit Plugin” GitHub Actions workflow, aligning the workflow contract with the current release flow and reducing stale secret surface area.

Changes:

  • Removed the FRAMER_ADMIN_SECRET entry from workflow_call.secrets in .github/workflows/submit-plugin.yml.
File summaries
File Description
.github/workflows/submit-plugin.yml Drops the unused FRAMER_ADMIN_SECRET secret declaration from the reusable workflow interface.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@xespona
xespona added this pull request to the merge queue Sep 9, 2026
Merged via the queue into main with commit a128bc8 Sep 9, 2026
12 checks passed
@xespona
xespona deleted the chore/drop-framer-admin-secret-declaration branch September 9, 2026 10:33
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.

3 participants