Skip to content

viewer: Add HTML helper to view nodes #1858

viewer: Add HTML helper to view nodes

viewer: Add HTML helper to view nodes #1858

Triggered via pull request February 19, 2024 14:27
Status Failure
Total duration 2m 20s
Artifacts

main.yml

on: pull_request
Matrix: check
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 12 warnings
Lint: api/db.py#L13
Unable to import 'kernelci.api.models'
Lint: api/main.py#L30
Unable to import 'kernelci.api.models'
Lint: tests/unit_tests/test_node_handler.py#L16
Unable to import 'kernelci.api.models'
check (3.10)
Process completed with exit code 4.
Lint: api/main.py#L723
TODO: We need to install templates UNDER api package
Lint: api/main.py#L726
Using open without explicitly specifying an encoding
Lint: tests/unit_tests/test_node_handler.py#L16
third party import "from kernelci.api.models import Node, Revision" should be placed before "from tests.unit_tests.conftest import BEARER_TOKEN"
Lint: tests/unit_tests/test_node_handler.py#L30
line too long (84 > 79 characters)
Lint: tests/unit_tests/test_node_handler.py#L43
unexpected spaces around keyword / parameter equals
Lint: tests/unit_tests/test_node_handler.py#L104
line too long (92 > 79 characters)
Lint: tests/unit_tests/test_node_handler.py#L122
line too long (92 > 79 characters)
Lint: tests/unit_tests/test_node_handler.py#L197
line too long (81 > 79 characters)
Lint: tests/unit_tests/test_node_handler.py#L208
unexpected spaces around keyword / parameter equals
Lint: tests/unit_tests/test_node_handler.py#L208
unexpected spaces around keyword / parameter equals
check (3.10)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
check (3.10)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/