Skip to content

Conversation

lumirlumir
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

I'm not sure if I'm the only one who feels this way, but I found that mixing relative and absolute import paths made things a bit messy. So, I've updated the import paths to use absolute paths for better readability and consistency.

I've also alphabetized the import order and arranged them from external dependencies to internal ones, which helps make everything look cleaner.

What changes did you make? (Give an overview)

In this PR, I've updated imports to use absolute paths for better readability

Related Issues

N/A

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

Actually, there are a few more modules where the import paths could be updated, but I intentionally excluded them to keep the scope manageable. For your reference, the ones not included are:

  • The ast, path, scope, and ui directories in src/components

Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit 8c3888c
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/68e8ff196d66430008c0cdd7
😎 Deploy Preview https://deploy-preview-182--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.

@eslintbot eslintbot added this to Triage Oct 10, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 10, 2025
@lumirlumir lumirlumir requested a review from a team October 10, 2025 12:42
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.

"I intentionally excluded some directories to keep the scope manageable." - sounds reasonable. If those folders (ast, path, scope, ui) are somewhat self-contained and internal to a larger module (e.g., all part of src/components/), keeping relative imports within them is totally fine. Leaving it open for a while in case others have any additional thoughts or opinions.

@amareshsm amareshsm added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 10, 2025
@amareshsm amareshsm moved this from Needs Triage to Implementing in Triage Oct 10, 2025
@amareshsm amareshsm merged commit e556097 into main Oct 14, 2025
9 checks passed
@amareshsm amareshsm deleted the refactor-update-imports-to-use-absolute-paths-for-better-readability branch October 14, 2025 09:57
@github-project-automation github-project-automation bot moved this from Implementing to Complete in Triage Oct 14, 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