Skip to content

Commit 31b662c

Browse files
feat: Transfer frontend-template-application improvements (#944)
* feat: Transfer frontend-template-application improvements * fix: fix disscusions and some refactor * fix: correct header and footer packeges placement * fix: fix spacing * fix: ci fix for stylelint * fix: add correct changes for packages * fix: fix lint errors * fix: add correct eslint config * fix: remove extra package and fix jest config * fix: update lock file
1 parent 2cadb33 commit 31b662c

File tree

15 files changed

+14588
-9033
lines changed

15 files changed

+14588
-9033
lines changed

.eslintignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
coverage/*
22
dist/
33
node_modules/
4-
jest.config.js
4+
jest.config.ts

.eslintrc.js

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,18 @@
11
// eslint-disable-next-line import/no-extraneous-dependencies
22
const { createConfig } = require('@openedx/frontend-build');
33

4-
module.exports = createConfig('eslint');
4+
module.exports = createConfig(
5+
'eslint',
6+
{
7+
rules: {
8+
'jsx-a11y/label-has-associated-control': [2, {
9+
controlComponents: ['Input'],
10+
}],
11+
'template-curly-spacing': 'off',
12+
'react-hooks/exhaustive-deps': 'off',
13+
'no-restricted-exports': 'off',
14+
// There is no reason to disallow this syntax anymore; we don't use regenerator-runtime in new browsers
15+
'no-restricted-syntax': 'off',
16+
},
17+
},
18+
);

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,3 +17,5 @@ temp/babel-plugin-react-intl
1717
*~
1818
/temp
1919
/.vscode
20+
21+
webpack.dev-tutor.config.js

.stylelintrc.json

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
{
2+
"extends": ["@edx/stylelint-config-edx"],
3+
"rules": {
4+
"selector-pseudo-class-no-unknown": [true, {
5+
"ignorePseudoClasses": ["export"]
6+
}],
7+
"unit-no-unknown": [true, {
8+
"ignoreUnits": ["\\.5"]
9+
}],
10+
"property-no-vendor-prefix": [true, {
11+
"ignoreProperties": ["animation", "filter", "transform", "transition"]
12+
}],
13+
"value-no-vendor-prefix": [true, {
14+
"ignoreValues": ["fill-available"]
15+
}],
16+
"function-no-unknown": null,
17+
"number-leading-zero": "never",
18+
"no-descending-specificity": null,
19+
"selector-class-pattern": null,
20+
"scss/no-global-function-names": null,
21+
"color-hex-case": "upper",
22+
"color-hex-length": "long",
23+
"scss/dollar-variable-empty-line-before": null,
24+
"scss/dollar-variable-colon-space-after": "at-least-one-space",
25+
"at-rule-no-unknown": null,
26+
"scss/at-rule-no-unknown": true,
27+
"scss/at-import-partial-extension": null,
28+
"scss/comment-no-empty": null,
29+
"import-notation": "string",
30+
"property-no-unknown": [true, {
31+
"ignoreProperties": ["xs", "sm", "md", "lg", "xl", "xxl"]
32+
}],
33+
"alpha-value-notation": "number"
34+
}
35+
}
Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
const { createConfig } = require('@openedx/frontend-build');
22

33
module.exports = createConfig('jest', {
4-
// setupFilesAfterEnv is used after the jest environment has been loaded. In general this is what you want.
5-
// If you want to add config BEFORE jest loads, use setupFiles instead.
4+
// setupFilesAfterEnv is used after the jest environment has been loaded. In general this is what you want.
5+
// If you want to add config BEFORE jest loads, use setupFiles instead.
66
setupFilesAfterEnv: [
77
'<rootDir>/src/setupTest.js',
88
],
@@ -11,3 +11,5 @@ module.exports = createConfig('jest', {
1111
'src/i18n',
1212
],
1313
});
14+
15+
export {};

0 commit comments

Comments
 (0)