Skip to content

Commit

Permalink
Merge remote-tracking branch 'upstream/main' into subtitles-as-tracks…
Browse files Browse the repository at this point in the history
…-with-tags
  • Loading branch information
Arnei committed Jun 21, 2023
2 parents 4e64e0d + 9e46141 commit 994ec1e
Show file tree
Hide file tree
Showing 68 changed files with 2,184 additions and 1,969 deletions.
48 changes: 48 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
module.exports = {
extends: [
"@opencast/eslint-config-ts-react",
"react-app/jest"
],
rules: {
// Semicolon usage is inconsistent right now. Both, "always" and "never",
// result in a significant number of warnings.
"semi": "off",

// Both kinds of quotes are used a lot
"quotes": "off",

// "always" gives 319 warning, "never" gives 991.
"object-curly-spacing": "off",

// Many multiline lists do not have a trailing comma right now
"comma-dangle": ["warn", "only-multiline"],

// Currently there are quite a lot of lines longer than 120 chars (85 ones).
// And 34 even longer than 140 chars.
"max-len": "off",

// Currently 120 warnings.
"@typescript-eslint/no-explicit-any": "off",

// `else` in new line is used often, and often an `if` condition is on the
// same line as `{`.
"brace-style": "off",

// A few of those are used.
"@typescript-eslint/no-non-null-assertion": "off",
},
overrides: [
{
files: ["./*.js"],
env: {
node: true,
},
},
{
files: ["./*.ts", "tests/**"],
parserOptions: {
project: false,
},
},
],
};
2 changes: 1 addition & 1 deletion .github/workflows/check-label.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
uses: jesusvasquez333/[email protected]
with:
github-token: '${{ secrets.GITHUB_TOKEN }}'
valid-labels: 'type:tests, type:infrastructure, type:enhancement, type:feature, type:dependencies, type:documentation, type:accessibility, type:security, type:bug, type:usability, type:visual-clarity'
valid-labels: 'type:tests, type:infrastructure, type:enhancement, type:feature, type:dependencies, type:documentation, type:accessibility, type:security, type:bug, type:usability, type:visual-clarity, type:code-quality'
invalid-labels: 'status:duplicate, status:conflicts, status:wontfix'
pull-request-number: '${{ github.event.pull_request.number }}'
disable-reviews: true
11 changes: 5 additions & 6 deletions config-overrides.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Overrides to make emotion css work with createReactApp
const { override, addBabelPreset } = require('customize-cra')

// Overrides to make emotion css work with createReactApp
const { override, addBabelPreset } = require('customize-cra')

module.exports = override(
addBabelPreset('@emotion/babel-preset-css-prop')
)
module.exports = override(
addBabelPreset('@emotion/babel-preset-css-prop')
)
Loading

0 comments on commit 994ec1e

Please sign in to comment.