Merge branch 'master' into feat/analytical-table-sticky-scrollbars

Author: Lukas742

.eslint-imports.js

@@ -1,6 +1,36 @@
 packages/*/src/lib/
-packages/*/lib/
-packages/*/dist/
+
+# build results for base package
+packages/base/cjs
+packages/base/Device
+packages/base/hooks
+packages/base/lib
+packages/base/polyfill
+packages/base/scripts
+packages/base/styling
+packages/base/types
+packages/base/utils
+
+# build results for charts package
+packages/charts/cjs
+packages/charts/components
+packages/charts/hooks
+packages/charts/interfaces
+packages/charts/internal
+packages/charts/lib
+
+# build results for main package
+packages/main/cjs
+packages/main/components
+packages/main/dist
+packages/main/enums
+packages/main/interfaces
+packages/main/internal
+packages/main/lib
+packages/main/scripts
+packages/main/webComponents
+
+
 scripts
 shared
 .eslintrc.js
@@ -11,3 +41,4 @@ rollup.config.js
 index.cjs.js
 index.esm.js
 packages/cra-template
+

.eslintignore

@@ -6,8 +6,10 @@ module.exports = {
     'plugin:@typescript-eslint/recommended',
     'plugin:@typescript-eslint/recommended-requiring-type-checking',
     'plugin:react/recommended',
-    'prettier',
-    './.eslint-imports.js'
+    'plugin:import/errors',
+    'plugin:import/warnings',
+    'plugin:import/typescript',
+    'prettier'
   ],
   parser: '@typescript-eslint/parser',
   parserOptions: {
@@ -122,6 +124,17 @@ module.exports = {
     'react/self-closing-comp': 'error',
     'spaced-comment': 'error',
     'use-isnan': 'error',
-    'valid-typeof': 'off'
+    'valid-typeof': 'off',
+    'import/order': [
+      'error',
+      {
+        'newlines-between': 'never',
+        alphabetize: {
+          order: 'asc',
+          caseInsensitive: true
+        }
+      }
+    ],
+    'import/no-duplicates': 2
   }
 };

.eslintrc.js

@@ -57,8 +57,8 @@
     "@types/jest": "^26.0.8",
     "@types/react": "^16.9.34",
     "@types/react-dom": "^16.9.6",
-    "@typescript-eslint/eslint-plugin": "^3.7.0",
-    "@typescript-eslint/parser": "^3.7.0",
+    "@typescript-eslint/eslint-plugin": "^4.4.0",
+    "@typescript-eslint/parser": "^4.4.0",
     "@ui5/webcomponents-tools": "^1.0.0-rc.8",
     "babel-loader": "^8.1.0",
     "chalk": "^4.0.0",
@@ -67,12 +67,12 @@
     "enzyme": "^3.11.0",
     "enzyme-adapter-react-16": "^1.15.2",
     "enzyme-to-json": "^3.4.4",
-    "eslint": "^7.5.0",
-    "eslint-config-prettier": "^6.11.0",
-    "eslint-plugin-import": "^2.22.0",
+    "eslint": "^7.10.0",
+    "eslint-config-prettier": "^6.12.0",
+    "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-prefer-arrow": "^1.2.2",
-    "eslint-plugin-react": "^7.20.3",
-    "eslint-plugin-react-hooks": "^4.0.8",
+    "eslint-plugin-react": "^7.21.3",
+    "eslint-plugin-react-hooks": "^4.1.2",
     "glob": "^7.1.6",
     "husky": "^4.2.5",
     "identity-obj-proxy": "^3.0.0",

config/rules/noIndexImportsRule.js

@@ -344,6 +344,13 @@ const getEventParameters = (parameters) => {
 
   const importStatements = [];
 
+  if (resolvedEventParameters.length === 0) {
+    return {
+      tsType: `(event: CustomEvent) => void`,
+      importStatements
+    };
+  }
+
   const detailPayload = resolvedEventParameters.map((parameter) => {
     if (parameter.importStatement) {
       importStatements.push(parameter.importStatement);

config/rules/noIndexImportsRule.ts

@@ -67,6 +67,9 @@ export const ColumnHeaderContainer = (props) => {
     <div {...headerProps} role="rowgroup" style={{ width: `${columnVirtualizer.totalSize}px` }}>
       {columnVirtualizer.virtualItems.map((virtualColumn: VirtualItem) => {
         const column = headerGroup.headers[virtualColumn.index];
+        if (!column) {
+          return null;
+        }
         const isLastColumn = !column.disableResizing && virtualColumn.index + 1 === headerGroup.headers.length;
         return (
           <>

package.json

@@ -179,6 +179,9 @@ export const VirtualTableBody = (props: VirtualTableBodyProps) => {
           >
             {columnVirtualizer.virtualItems.map((virtualColumn) => {
               const cell = row.cells[virtualColumn.index];
+              if (!cell) {
+                return null;
+              }
               const cellProps = cell.getCellProps();
               if (row.original?.emptyRow) {
                 return <div {...cellProps} />;

packages/main/scripts/create-web-components-wrapper.js

@@ -33,7 +33,7 @@ export interface ButtonPropTypes extends Omit<WithWebComponentPropTypes, 'onClic
   /**
    * Fired when the <code>Button</code> is activated either with a mouse/tap or by using the Enter or Space key. <br><br> <b>Note:</b> The event will not be fired if the <code>disabled</code> property is set to <code>true</code>.
    */
-  onClick?: (event: CustomEvent<{}>) => void;
+  onClick?: (event: CustomEvent) => void;
 }
 
 /**