Skip to content

chore: delete .github/PULL_REQUEST_TEMPLATE/atomic-stencil-lit-migrat…#7184

Open
louis-bompart wants to merge 1 commit intomainfrom
louis-bompart-patch-3
Open

chore: delete .github/PULL_REQUEST_TEMPLATE/atomic-stencil-lit-migrat…#7184
louis-bompart wants to merge 1 commit intomainfrom
louis-bompart-patch-3

Conversation

@louis-bompart
Copy link
Collaborator

…ion.md

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR deletes the PR template file for Stencil to Lit component migrations in the Atomic package. The template contained a checklist for ensuring migrated components meet quality standards including testing, documentation, accessibility, and API compatibility requirements.

Changes:

  • Removes the atomic-stencil-lit-migration.md PR template that was used to guide Stencil to Lit component migrations
Comments suppressed due to low confidence (1)

.github/PULL_REQUEST_TEMPLATE/atomic-stencil-lit-migration.md:1

  • Deleting this PR template will break the GitHub workflow .github/workflows/update-pr-template-lit-migration.yml which attempts to read this file at line 31. The workflow runs when the 'new-lit-component' label is added to a PR and will fail with a "file not found" error.

Additionally, this template is referenced in two migration agent documentation files:

  • .github/agents/stencil-to-lit-migration-v1.agent.md (line 175)
  • .github/agents/stencil-to-lit-migration-v2.agent.md (line 400)

Before deleting this template, you should:

  1. Either update or remove the workflow file .github/workflows/update-pr-template-lit-migration.yml
  2. Update the agent documentation files to remove or replace references to this template

If the Stencil to Lit migration is complete and this template is no longer needed, these supporting files should be cleaned up in the same PR.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants