Skip to content

ci: disable release drafter autolabeler - #107

Closed
KooshaPari wants to merge 7 commits into
mainfrom
ci/repair-release-drafter-base
Closed

ci: disable release drafter autolabeler#107
KooshaPari wants to merge 7 commits into
mainfrom
ci/repair-release-drafter-base

Conversation

@KooshaPari

Copy link
Copy Markdown
Owner

Removes the Release Drafter Auto Labeler from the generated pull_request_target workflow.

The failed #106 job loads the target branch workflow and cannot be repaired by changes on #106 itself; this base-branch repair prevents the denied label write.

Validation:

  • cargo test -p forge_ci test_release_drafter -- --exact --nocapture
  • CI=1 cargo test -p forge_ci test_release_drafter -- --exact --nocapture
  • cargo fmt --check --package forge_ci
  • actionlint .github/workflows/release-drafter.yml

@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6cdd0d63c8

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +17 to +20
assert!(
!workflow.contains("Auto Labeler"),
"pull_request_target must not execute label writes"
);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Move this regression test next to its source

This new regression assertion is being added in the integration test file, but the repository guideline requires tests to live in the same file as the source they cover. Please move this coverage into a #[cfg(test)] module next to generate_release_drafter_workflow or draft_release_update_job so the release-drafter invariant stays with the code it verifies.

AGENTS.md reference: AGENTS.md:L42-L42

Useful? React with 👍 / 👎.

@socket-security

socket-security Bot commented Jul 30, 2026

Copy link
Copy Markdown

Dependency limit exceeded — report not shown.

This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report.

Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard.

Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account.

@mergify mergify Bot added the rust label Jul 31, 2026
@mergify

mergify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@github-actions github-actions Bot removed the rust label Jul 31, 2026
@mergify mergify Bot added the rust label Jul 31, 2026
@KooshaPari

Copy link
Copy Markdown
Owner Author

Reconciled against fork main 729f693 after PR #110 and #106 merges. Rebase of this branch skipped all previously applied workflow/forge_ci commits; resulting tree is byte-identical to main (0 commits ahead, 0 behind; empty diff). Closing as fully superseded.

@KooshaPari

Copy link
Copy Markdown
Owner Author

Closed: fully superseded by merged workflow repairs on main.

@KooshaPari KooshaPari closed this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant