Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0047890
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] A. Platformminorhave not tried2021-10-20 09:372022-02-01 08:07
ReporterjarmendarizView Statuspublic 
Assigned ToTriage Platform Base 
PrioritynormalResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0047890: Upgrade ESLint to version 8.x.x

DescriptionRecently a new major version of ESLint have been published. Amont many improvements, bugfixes, there are a number of breaking changes listed here https://eslint.org/docs/8.0.0/user-guide/migrating-to-8.0.0 [^]

Steps To ReproduceUpgrade ESLint and related plugins to their latest version and notice the new errors that appear when running the jslint script.

At the moment of writing, running this script in a POS2 environment the following errors appears:

- Most modules has errors related to tests (jest/prefer-to-be, jest/expect-expect, jest/valid-title)

- POS2 module shows this error message in several files:

Parsing error: Must use import to load ES Module: /home/openbravo/Projects/pos2/openbravo/node_modules/eslint/node_modules/eslint-scope/lib/definition.js
require() of ES modules is not supported.
require() of /home/openbravo/Projects/pos2/openbravo/node_modules/eslint/node_modules/eslint-scope/lib/definition.js from /home/openbravo/Projects/pos2/openbravo/node_modules/babel-eslint/lib/require-from-eslint.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename definition.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /home/openbravo/Projects/pos2/openbravo/node_modules/eslint/node_modules/eslint-scope/package.json

- The following error appears in core2:

Error: Rules with suggestions must set the `meta.hasSuggestions` property to `true`.
Occurred while linting /home/openbravo/Projects/pos2/openbravo/modules/org.openbravo.core2/web-jspack/org.openbravo.core2/src/components/BaseKeymap/stories/BaseKeymap.stories.jsx:142
Rule: "react-hooks/exhaustive-deps"
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
There are no notes attached to this issue.

- Issue History
Date Modified Username Field Change
2021-10-20 09:37 jarmendariz New Issue
2021-10-20 09:37 jarmendariz Assigned To => platform
2021-10-20 09:37 jarmendariz Modules => Core
2021-10-20 09:37 jarmendariz Triggers an Emergency Pack => No
2022-02-01 08:07 alostale Assigned To platform => Triage Platform Base


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker