Skip to content

Scope Martian triage to bug-labeled issues for jlowin#3124

Merged
jlowin merged 1 commit intomainfrom
workflow-triage-update
Feb 9, 2026
Merged

Scope Martian triage to bug-labeled issues for jlowin#3124
jlowin merged 1 commit intomainfrom
workflow-triage-update

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Feb 9, 2026

Strawgate-opened issues still always trigger Martian triage. For jlowin-opened issues, triage now only runs when the issue has a bug label, avoiding unnecessary triage on feature discussions and other non-bug issues.

@jlowin jlowin merged commit dfb857a into main Feb 9, 2026
12 checks passed
@jlowin jlowin deleted the workflow-triage-update branch February 9, 2026 15:35
@marvin-context-protocol marvin-context-protocol Bot added the enhancement Improvement to existing functionality. For issues and smaller PR improvements. label Feb 9, 2026
Copy link
Copy Markdown

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

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: 1f8d5ef825

ℹ️ 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".

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

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +11 to 13
(github.event.action == 'opened' && github.actor == 'strawgate') ||
(github.event.action == 'opened' && github.actor == 'jlowin' && contains(github.event.issue.labels.*.name, 'bug')) ||
(github.event.action == 'labeled' && github.event.label.name == 'triage-martian' && contains(fromJSON('["OWNER", "MEMBER", "COLLABORATOR"]'), github.event.sender.author_association))
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Trigger triage when bug label is added later

As written, the job only runs for opened events where the bug label is already present. If jlowin opens an issue without the label and it’s applied later by a labeler or maintainer (a common workflow), the labeled event won’t trigger triage because it only checks for triage-martian. That means legitimate bug issues opened by jlowin will skip triage entirely unless the label is set at creation. Consider handling the labeled event for bug as well if the intent is “triage whenever a jlowin issue has a bug label.”

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improvement to existing functionality. For issues and smaller PR improvements.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant