Skip to content
PR Review

Removing withStreams. #9127

Sign in to view logs

Removing withStreams.

Removing withStreams. #9127

Triggered via pull request June 13, 2024 10:03
@dennisoelkersdennisoelkers
opened #19631
refactor/remove-with-streams
Status Failure
Total duration 5m 24s
Artifacts –

reviewbot.yml

on: pull_request
Reviewbot
5m 16s
Reviewbot
Fit to window
Zoom out
Zoom in

Annotations

1 error and 10 warnings
Reviewbot
Found 8 linter hints in the changed code.
Reviewbot
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-java@v3, Graylog2/reviewbot@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Reviewbot
The following actions uses node12 which is deprecated and will be forced to run on node16: Graylog2/reviewbot@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
There should be at least one empty line between import groups: graylog2-web-interface/src/components/event-definitions/event-definition-types/FilterAggregationFormContainer.tsx#L23
No further rule information available.
`contexts/StreamsContext` import should occur before import of `./FilterAggregationForm`: graylog2-web-interface/src/components/event-definitions/event-definition-types/FilterAggregationFormContainer.tsx#L24
No further rule information available.
Expected blank line before this statement.: graylog2-web-interface/src/components/event-definitions/event-definition-types/FilterAggregationFormContainer.tsx#L30
See https://eslint.org/docs/rules/padding-line-between-statements for details.
There should be at least one empty line between import groups: graylog2-web-interface/src/components/event-definitions/event-definition-types/bindings.ts#L20
No further rule information available.
All imports in the declaration are only used as types. Use `import type`.: graylog2-web-interface/src/components/event-definitions/event-definition-types/bindings.ts#L21
`graylog-web-plugin/plugin` import should occur before import of `./FilterAggregationFormContainer`: graylog2-web-interface/src/components/event-definitions/event-definition-types/bindings.ts#L21
No further rule information available.
A space is required after '{'.: graylog2-web-interface/src/components/event-definitions/event-definition-types/bindings.ts#L21
See https://eslint.org/docs/rules/object-curly-spacing for details.
A space is required before '}'.: graylog2-web-interface/src/components/event-definitions/event-definition-types/bindings.ts#L21
See https://eslint.org/docs/rules/object-curly-spacing for details.