Skip to content

Conversation

@lumirlumir
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

Hi,

This PR is part of the process to centralize workflows. In this PR, I've added the add-to-triage.yml workflow, which uses the centralized workflow.

You can find more details about this workflow in eslint/workflows#4 and eslint/workflows#7.

What changes did you make? (Give an overview)

Related Issues

Is there anything you'd like reviewers to focus on?

@lumirlumir
Copy link
Member Author

The workflow is currently working as expected.

https://github.com/eslint/generator-eslint/actions/runs/16748997149

image

@lumirlumir
Copy link
Member Author

lumirlumir commented Aug 5, 2025

Hmm, the CI failed, but I don't think the failure is related to this change 😅


I’ve updated the yeoman dependencies to the latest versions, which resolved the CI error.

Should this version bump be handled in a separate PR? If so, I’d be happy to create one.

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

LGTM. Approving for merge.

@nzakas nzakas merged commit 93add84 into main Aug 5, 2025
11 checks passed
@nzakas nzakas deleted the lumirlumir-patch-1 branch August 5, 2025 14:21
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

2 participants