Skip to content

feat: change tool name @W-21858914@#434

Merged
iowillhoit merged 7 commits intomainfrom
change-accessibility
Apr 3, 2026
Merged

feat: change tool name @W-21858914@#434
iowillhoit merged 7 commits intomainfrom
change-accessibility

Conversation

@pakusruti
Copy link
Copy Markdown
Collaborator

What does this PR do?

@W-21858914@

  • Change guide_lwc_accessibility tool name to guide_component_accessibility

What issues does this PR fix or reference?

@pakusruti pakusruti marked this pull request as ready for review April 1, 2026 19:49
@pakusruti pakusruti requested review from a team as code owners April 1, 2026 19:49
@pakusruti pakusruti marked this pull request as draft April 1, 2026 23:51
if(invalidTools.includes('guide_lwc_accessibility')) {
ux.stderr('Tool "guide_lwc_accessibility" has been renamed to "guide_component_accessibility". Update config to remove this warning.')
// Remove that entry from invalidTools
invalidTools.splice(invalidTools.indexOf('guide_component_accessibility'), 1);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
invalidTools.splice(invalidTools.indexOf('guide_component_accessibility'), 1);
invalidTools.splice(invalidTools.indexOf('guide_lwc_accessibility'), 1);

This needs to be the old tool name.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah I updated in my local forgot to push. It is updated now.

@pakusruti pakusruti marked this pull request as ready for review April 2, 2026 20:44
@pakusruti pakusruti requested a review from iowillhoit April 2, 2026 20:44
@iowillhoit iowillhoit merged commit b34e830 into main Apr 3, 2026
11 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