Skip to content

Conversation

lumirlumir
Copy link
Member

@lumirlumir lumirlumir commented Aug 30, 2025

Prerequisites checklist

What is the purpose of this pull request?

In this PR, I've made a small change.

LabeledSelect.tsx was the only file not using kebab-case for its file name, so I renamed it to use kebab-case for consistency.

All other files in the Code Explorer follow kebab-case for file names. So, this change would improve consistency across the code base.

What changes did you make? (Give an overview)

LabeledSelect.tsx was the only file not using kebab-case for its file name, so I renamed it to use kebab-case for consistency.

Related Issues

N/A

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

N/A

@eslintbot eslintbot added this to Triage Aug 30, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Aug 30, 2025
Copy link

netlify bot commented Aug 30, 2025

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit 4da6621
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/68b6bfa45b6ab500080e4af9
😎 Deploy Preview https://deploy-preview-130--eslint-code-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

amareshsm
amareshsm previously approved these changes Sep 1, 2025
Copy link
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks.

@amareshsm
Copy link
Member

Can you pls resolve the conflicts?

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Sep 2, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Triaging in Triage Sep 2, 2025
@lumirlumir lumirlumir moved this from Triaging to Implementing in Triage Sep 2, 2025
@lumirlumir
Copy link
Member Author

@amareshsm Sure! I've resolved merge conflicts.

Copy link
Member

@amareshsm amareshsm left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks.

@amareshsm amareshsm merged commit 246d5b3 into main Sep 2, 2025
8 checks passed
@amareshsm amareshsm deleted the chore-use-kabab-case-in-file-name branch September 2, 2025 19:05
@github-project-automation github-project-automation bot moved this from Implementing to Complete in Triage Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion chore
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

2 participants