GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||
GeezerVison is showing github.com — `no-redeclare` loose parser tests · Issue #14315 · eslint/eslint Closed Labels acceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusionarchived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussionchoreThis change is not user-facingThis change is not user-facing DescriptionThe version of ESLint you are using. v7.24.0 The problem you want to solve. We have several tests for the eslint/tests/lib/rules/no-redeclare.js Lines 208 to 216 in 555cbf3
This is normally a syntax error reported by the parser, so the tests are using fixture parser tests/tools/loose-parser.js The fixture parser requires an internal module from eslint/tests/tools/loose-parser.js Line 10 in 555cbf3
The problem is - after we release eslint/js#469, that module will be no more available as it isn't defined by Your take on the correct solution to problem. Just remove those tests? Core rules are not generally designed to handle code that has syntax errors, at least for now. Are you willing to submit a pull request to implement this change? Yes. Reacted by 唯然, Nicholas C. Zakas and Brandon Mills Activity
MetadataMetadataAssigneesLabelsacceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusionarchived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussionchoreThis change is not user-facingThis change is not user-facing TypeProjects
MilestoneRelationshipsDevelopmentIssue actions | ||||||||||||||||||||
| You are visitor 01402520 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |