Skip to content

Code review fix

Code review fix #948

Triggered via push July 19, 2024 19:23
Status Success
Total duration 12m 57s
Artifacts
javascript_lint_and_tests
12m 50s
javascript_lint_and_tests
Fit to window
Zoom out
Zoom in

Annotations

33 warnings and 1 notice
[vale] README.npm.md#L11: README.npm.md#L11
[Kedro-viz.weaselwords] 'Beautifully' is a weasel word!
[vale] README.npm.md#L23: README.npm.md#L23
[Kedro-viz.Spellings] Did you really mean 'npm'?
[vale] README.npm.md#L41: README.npm.md#L41
[Kedro-viz.Spellings] Did you really mean 'Autoreload'?
[vale] README.npm.md#L43: README.npm.md#L43
[Kedro-viz.weaselwords] 'Many' is a weasel word!
[vale] README.npm.md#L154: README.npm.md#L154
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L156: README.npm.md#L156
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L157: README.npm.md#L157
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L158: README.npm.md#L158
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L159: README.npm.md#L159
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L160: README.npm.md#L160
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L161: README.npm.md#L161
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L162: README.npm.md#L162
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L163: README.npm.md#L163
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L164: README.npm.md#L164
[Kedro-viz.Spellings] Did you really mean 'boolean'?
[vale] README.npm.md#L175: README.npm.md#L175
[Kedro-viz.headings] 'Standalone Example Repository' should use sentence-style capitalization.
[vale] README.npm.md#L177: README.npm.md#L177
[Kedro-viz.toowordy] 'demonstrate' is too wordy
[vale] README.npm.md#L190: README.npm.md#L190
[Kedro-viz.headings] 'Feature Flags' should use sentence-style capitalization.
[vale] README.npm.md#L196: README.npm.md#L196
[Kedro-viz.Spellings] Did you really mean 'sizewarning'?
[vale] README.npm.md#L196: README.npm.md#L196
[Kedro-viz.words] Use '' instead of 'very'.
[vale] README.npm.md#L196: README.npm.md#L196
[Kedro-viz.weaselwords] 'very' is a weasel word!
[vale] README.npm.md#L201: README.npm.md#L201
[Kedro-viz.weaselwords] 'currently' is a weasel word!
[vale] README.npm.md#L205: README.npm.md#L205
[Kedro-viz.words] Use 'few', 'several', or 'many' instead of 'a number of'.
[vale] README.npm.md#L205: README.npm.md#L205
[Kedro-viz.toowordy] 'a number of' is too wordy
[vale] README.npm.md#L209: README.npm.md#L209
[Kedro-viz.words] Use '' instead of 'please'.
[vale] README.npm.md#L217: README.npm.md#L217
[Kedro-viz.Spellings] Did you really mean 'Zenodo'?
[vale] docs/source/kedro-viz_visualisation.md#L241: docs/source/kedro-viz_visualisation.md#L241
[Kedro-viz.words] Use '' instead of 'please'.
[vale] docs/source/kedro-viz_visualisation.md#L241: docs/source/kedro-viz_visualisation.md#L241
[Kedro-viz.words] Use 'see', 'read', or 'follow' instead of 'refer to'.
[vale] docs/source/kedro-viz_visualisation.md#L241: docs/source/kedro-viz_visualisation.md#L241
[Kedro-viz.Spellings] Did you really mean 'npm'?
javascript_lint_and_tests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
javascript_lint_and_tests
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/cache@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
javascript_lint_and_tests: src/components/app/app.test.js#L15
'prettifyName' is defined but never used
javascript_lint_and_tests: src/selectors/nodes.test.js#L34
'getNodeName' is assigned a value but never used
javascript_lint_and_tests: src/selectors/nodes.test.js#L35
'getNodeFullName' is assigned a value but never used
[vale] README.npm.md#L113: README.npm.md#L113
[Kedro-viz.sentencelength] Try to keep your sentence length to 30 words or fewer.