chore: remove unused JS files docs-main.js and navigation.js#1024
chore: remove unused JS files docs-main.js and navigation.js#1024amareshsm wants to merge 2 commits into
Conversation
✅ Deploy Preview for hi-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for new-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for es-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for ja-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for zh-hans-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for de-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for fr-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for pt-br-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Removes two legacy JavaScript asset files that are no longer referenced by the site templates or build pipeline, helping reduce dead code in the repository.
Changes:
- Deleted
src/assets/js/docs-main.js. - Deleted
src/assets/js/navigation.js.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/assets/js/docs-main.js | Removed unused legacy docs-related UI scripts. |
| src/assets/js/navigation.js | Removed unused legacy navigation script. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lumirlumir
left a comment
There was a problem hiding this comment.
Could the following .docs-main classes also be removed?
I'm not 100% sure, but I can't find any references to this class across the codebases.
Looking through the codebase history, this class appears to be a leftover from https://github.com/eslint/eslint.org/pull/96/changes#diff-6f5ffd52cd13a409b0e4cad66d8414be4140b29ded5b2cdbbd84cedce9617e28.
eslint.org/src/assets/scss/foundations.scss
Line 290 in 1df2fdc
eslint.org/src/assets/scss/foundations.scss
Line 292 in 1df2fdc
eslint.org/src/assets/scss/foundations.scss
Line 394 in 1df2fdc
eslint.org/src/assets/scss/components/ads.scss
Lines 36 to 38 in 1df2fdc
Prerequisites checklist
AI acknowledgment
What is the purpose of this pull request?
remove unused JS files docs-main.js and navigation.js
What changes did you make? (Give an overview)
remove unused JS files docs-main.js and navigation.js
Related Issues
Is there anything you'd like reviewers to focus on?