Skip to content

Add workflow#2488

Merged
Krishanx92 merged 1 commit into
wso2:mainfrom
Krishanx92:cli
Jul 7, 2026
Merged

Add workflow#2488
Krishanx92 merged 1 commit into
wso2:mainfrom
Krishanx92:cli

Conversation

@Krishanx92

Copy link
Copy Markdown
Contributor

Purpose

Explain why this feature or fix is required. Describe the underlying problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe what solutions this feature or fix introduces to address the problems outlined above.

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI. Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Automation tests

  • Unit tests

    Code coverage information

  • Integration tests

    Details about the test cases and coverage

Security checks

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 532e80dd-4339-47f3-bb7d-a00afbe01337

📥 Commits

Reviewing files that changed from the base of the PR and between d99d49a and 4bbe726.

📒 Files selected for processing (2)
  • .github/workflows/design-done-label.yml
  • .github/workflows/enforce-design-gate.yml

📝 Walkthrough

Walkthrough

Modifies the design-done-label workflow to bidirectionally add/remove the "Design Done" label based on checklist state and adds new logic for a "Mail Sent" label tied to a "Design Mail" checklist item. Introduces a new workflow, "Enforce Design Gate," that posts a warning comment when required labels are removed from an issue.

Changes

Design gate labeling workflows

Layer / File(s) Summary
Design Done label bidirectional handling
.github/workflows/design-done-label.yml
Updates the permissions comment and changes checklist handling so the "Design Done" label is added when the "Design Document" item is checked and removed when unchecked, with error handling for missing labels.
Mail Sent label handling for Design Mail checklist
.github/workflows/design-done-label.yml
Adds a new step managing a "Mail Sent" label based on the "Design Mail" checklist item, creating the label if missing, adding it when checked, and removing it when unchecked.
Enforce design gate warning workflow
.github/workflows/enforce-design-gate.yml
Adds a new workflow triggered on label add/remove events that posts a warning comment when "Design Done" or "Mail Sent" is removed from an issue, listing the required labels.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant GitHubIssue
  participant EnforceDesignGate

  User->>GitHubIssue: Remove "Design Done" or "Mail Sent" label
  GitHubIssue->>EnforceDesignGate: unlabeled event
  EnforceDesignGate->>EnforceDesignGate: Check if removed label is required
  EnforceDesignGate->>GitHubIssue: Post warning comment
Loading

Suggested labels: ci, github-actions

Suggested reviewers: none identified

Poem:
A checkbox ticked, a label springs to life,
Unchecked again, it vanishes like strife.
A warning now stands guard at the gate,
"Design's not done — please don't move too late!"
🐰 Hopping through workflows, keeping issues straight.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Krishanx92
Krishanx92 merged commit 2e61dc2 into wso2:main Jul 7, 2026
3 of 5 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Jul 7, 2026
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