Skip to content

Update workflow#2508

Merged
Krishanx92 merged 2 commits into
wso2:mainfrom
Krishanx92:cli
Jul 7, 2026
Merged

Update workflow#2508
Krishanx92 merged 2 commits 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: 6bf5c103-a722-41e9-94dc-53737dbfbb74

📥 Commits

Reviewing files that changed from the base of the PR and between a73f12e and 22b25ba.

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

📝 Walkthrough

Walkthrough

The enforce-design-gate.yml workflow is migrated from a Projects v1 card-event trigger to a scheduled GitHub Actions job using github-script and GraphQL. It discovers Projects v2, locates Status fields/options, paginates project items, and enforces design-gate labels by reverting Status and commenting on non-compliant issues.

Changes

Design Gate Workflow Migration

Layer / File(s) Summary
Trigger and permissions update
.github/workflows/enforce-design-gate.yml
Switches from project_card event trigger to daily cron/workflow_dispatch, updates permissions to issues: write and repository-projects: write, uses built-in GITHUB_TOKEN.
Projects v2 discovery and field mapping
.github/workflows/enforce-design-gate.yml
Adds GraphQL queries to find org/user-owned Projects v2, resolve the Status field, map Development/Design options, and fail if none found.
Paginated project item retrieval
.github/workflows/enforce-design-gate.yml
Fetches project items in paginated batches, including issue content, labels, and Status field values.
Design gate enforcement and reporting
.github/workflows/enforce-design-gate.yml
Checks feature issues in Development for required labels (Design Done, Mail Sent), reverts Status to Design via mutation, posts caution comments, and logs checked/reverted counts.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Possibly related PRs

  • wso2/api-platform#2488: Both PRs modify the same enforce-design-gate.yml workflow logic enforcing design-gate labels via comments.
  • wso2/api-platform#2498: Both PRs modify enforce-design-gate.yml to enforce the same Development status gate and required labels.

Suggested reviewers: RakhithaRR, Tharsanan1, VirajSalaka, renuka-fernando, tharindu1st, malinthaprasan

✨ 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 a676c39 into wso2:main Jul 7, 2026
1 of 6 checks passed
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.

2 participants