-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'main' into 1250-ui-language-selector
# Conflicts: # lib/platform-bible-react/dist/index.cjs # lib/platform-bible-react/dist/index.cjs.map # lib/platform-bible-react/dist/index.d.ts # lib/platform-bible-react/dist/index.js # lib/platform-bible-react/dist/index.js.map # lib/platform-bible-react/src/index.ts
- Loading branch information
Showing
122 changed files
with
61,518 additions
and
3,804 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -61,6 +61,7 @@ | |
| "newtonsoft", | ||
| "nodebuffer", | ||
| "nums", | ||
| "openrpc", | ||
| "papi", | ||
| "papis", | ||
| "paranext", | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| root = true | ||
|
|
||
| [*] | ||
| indent_style = space | ||
| indent_size = 2 | ||
| end_of_line = lf | ||
| charset = utf-8 | ||
| trim_trailing_whitespace = true | ||
| insert_final_newline = true | ||
| max_line_length = 100 | ||
|
|
||
| [*.md] | ||
| trim_trailing_whitespace = false | ||
|
|
||
| [*.cs] | ||
| indent_size = 4 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| # #region shared with https://github.com/paranext/paranext-multi-extension-template/blob/main/.eslintignore | ||
|
|
||
| # Please keep this file in sync with .prettierignore and .stylelintignore | ||
|
|
||
| # Logs | ||
| logs | ||
| *.log | ||
|
|
||
| # Runtime data | ||
| pids | ||
| *.pid | ||
| *.seed | ||
|
|
||
| # Dependency directory | ||
| # https://www.npmjs.org/doc/misc/npm-faq.html#should-i-check-my-node_modules-folder-into-git | ||
| node_modules | ||
|
|
||
| # OSX | ||
| .DS_Store | ||
|
|
||
| .idea | ||
| npm-debug.log.* | ||
| *.css.d.ts | ||
| *.sass.d.ts | ||
| *.scss.d.ts | ||
|
|
||
| # Built files | ||
| dist | ||
| release | ||
| temp-build | ||
|
|
||
| # generated files | ||
| package-lock.json | ||
|
|
||
| # #endregion |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,170 @@ | ||
| // #region shared with https://github.com/paranext/paranext-multi-extension-template/blob/main/.eslintrc.cjs | ||
|
|
||
| module.exports = { | ||
| extends: [ | ||
| // https://github.com/electron-react-boilerplate/eslint-config-erb/blob/main/index.js | ||
| // airbnb rules are embedded in erb https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb | ||
| 'erb', | ||
| // Make sure this is last so it gets the chance to override other configs. | ||
| // See https://github.com/prettier/eslint-config-prettier and https://github.com/prettier/eslint-plugin-prettier | ||
| 'plugin:prettier/recommended', | ||
| ], | ||
|
|
||
| rules: { | ||
| // Some rules in this following shared region are not applied since they are overridden in subsequent regions | ||
| // #region shared with https://github.com/paranext/paranext-core/blob/main/.eslintrc.js except certain overrides | ||
|
|
||
| // #region ERB rules | ||
|
|
||
| // Use `noImplicitReturns` instead. See https://typescript-eslint.io/rules/consistent-return/. | ||
| 'consistent-return': 'off', | ||
| 'import/extensions': 'off', | ||
| // A temporary hack related to IDE not resolving correct package.json | ||
| 'import/no-extraneous-dependencies': 'off', | ||
| 'import/no-import-module-exports': 'off', | ||
| 'import/no-unresolved': 'error', | ||
| 'react/jsx-filename-extension': 'off', | ||
| 'react/react-in-jsx-scope': 'off', | ||
|
|
||
| // #endregion | ||
|
|
||
| // #region Platform.Bible rules | ||
|
|
||
| // Rules in each section are generally in alphabetical order. However, several | ||
| // `@typescript-eslint` rules require disabling the equivalent ESLint rule. So in these cases | ||
| // each ESLint rule is turned off immediately above the corresponding `@typescript-eslint` rule. | ||
| 'class-methods-use-this': 'off', | ||
| '@typescript-eslint/class-methods-use-this': [ | ||
| 'error', | ||
| { ignoreOverrideMethods: true, ignoreClassesThatImplementAnInterface: false }, | ||
| ], | ||
| '@typescript-eslint/explicit-member-accessibility': ['error', { accessibility: 'no-public' }], | ||
| 'lines-between-class-members': 'off', | ||
| '@typescript-eslint/lines-between-class-members': [ | ||
| 'error', | ||
| 'always', | ||
| { exceptAfterSingleLine: true, exceptAfterOverload: true }, | ||
| ], | ||
| '@typescript-eslint/member-ordering': 'error', | ||
| 'no-empty-function': 'off', | ||
| '@typescript-eslint/no-empty-function': [ | ||
| 'error', | ||
| { | ||
| allow: ['arrowFunctions', 'functions', 'methods'], | ||
| }, | ||
| ], | ||
| '@typescript-eslint/no-explicit-any': 'error', | ||
| 'no-redeclare': 'off', | ||
| '@typescript-eslint/no-redeclare': 'error', | ||
| 'no-restricted-imports': [ | ||
| 'error', | ||
| { | ||
| patterns: [ | ||
| { | ||
| group: ['shared/*', 'renderer/*', 'extension-host/*', 'node/*', 'client/*', 'main/*'], | ||
| message: `Importing from this path is not allowed. Try importing from @papi/core. Imports from paths like 'shared', 'renderer', 'node', 'client' and 'main' are not allowed to prevent unnecessary import break.`, | ||
| }, | ||
| ], | ||
| }, | ||
| ], | ||
| 'no-shadow': 'off', | ||
| '@typescript-eslint/no-shadow': 'error', | ||
| 'no-use-before-define': 'off', | ||
| '@typescript-eslint/no-use-before-define': [ | ||
| 'error', | ||
| { functions: false, allowNamedExports: true, typedefs: false, ignoreTypeReferences: true }, | ||
| ], | ||
| '@typescript-eslint/no-unnecessary-type-assertion': 'error', | ||
| 'no-unused-vars': 'off', | ||
| '@typescript-eslint/no-unused-vars': 'error', | ||
| 'no-useless-constructor': 'off', | ||
| '@typescript-eslint/no-useless-constructor': 'error', | ||
| 'comma-dangle': ['error', 'always-multiline'], | ||
| 'import/no-anonymous-default-export': ['error', { allowCallExpression: false }], | ||
| indent: 'off', | ||
| 'jsx-a11y/label-has-associated-control': [ | ||
| 'error', | ||
| { | ||
| assert: 'either', | ||
| }, | ||
| ], | ||
| // Should use our logger anytime you want logs that persist. Otherwise use console only in testing | ||
| 'no-console': 'warn', | ||
| 'no-null/no-null': 2, | ||
| 'no-plusplus': ['error', { allowForLoopAfterthoughts: true }], | ||
| 'no-type-assertion/no-type-assertion': 'error', | ||
| 'prettier/prettier': ['warn', { tabWidth: 2, trailingComma: 'all' }], | ||
| 'react/jsx-indent-props': ['warn', 2], | ||
| 'react/jsx-props-no-spreading': ['error', { custom: 'ignore' }], | ||
| 'react/require-default-props': 'off', | ||
|
|
||
| // #endregion | ||
|
|
||
| // #endregion | ||
|
|
||
| // #region Overrides to rules from paranext-core | ||
|
|
||
| 'import/no-unresolved': ['error', { ignore: ['@papi'] }], | ||
|
|
||
| // #endregion | ||
| }, | ||
| globals: { | ||
| globalThis: 'readonly', | ||
| }, | ||
| overrides: [ | ||
| { | ||
| // Allow this file to have overrides to rules from paranext-core | ||
| files: ['.eslintrc.*js'], | ||
| rules: { | ||
| 'no-dupe-keys': 'off', | ||
| }, | ||
| }, | ||
| { | ||
| files: ['*.js'], | ||
| rules: { | ||
| strict: 'off', | ||
| }, | ||
| }, | ||
| { | ||
| // Don't require extensions to have a default export for "activate()" | ||
| files: ['*.ts'], | ||
| rules: { | ||
| 'import/prefer-default-export': 'off', | ||
| }, | ||
| }, | ||
| { | ||
| files: ['./lib/*', './webpack/*'], | ||
| rules: { | ||
| // These files are scripts not running in Platform.Bible, so they can't use the logger | ||
| 'no-console': 'off', | ||
| }, | ||
| }, | ||
| { | ||
| files: ['*.d.ts'], | ||
| rules: { | ||
| // Allow .d.ts files to self import so they can refer to their types in `papi-shared-types` | ||
| 'import/no-self-import': 'off', | ||
| }, | ||
| }, | ||
| ], | ||
| parserOptions: { | ||
| ecmaVersion: 2022, | ||
| sourceType: 'module', | ||
| project: './tsconfig.lint.json', | ||
| tsconfigRootDir: __dirname, | ||
| createDefaultProgram: true, | ||
| }, | ||
| plugins: ['@typescript-eslint', 'no-type-assertion', 'no-null'], | ||
| settings: { | ||
| 'import/resolver': { | ||
| typescript: { | ||
| alwaysTryTypes: true, | ||
| }, | ||
| }, | ||
| 'import/parsers': { | ||
| '@typescript-eslint/parser': ['.ts', '.tsx'], | ||
| }, | ||
| }, | ||
| }; | ||
|
|
||
| // #endregion |
Oops, something went wrong.