Skip to content

[PPP-502] TKMS - Add decorator for message headers #208

[PPP-502] TKMS - Add decorator for message headers

[PPP-502] TKMS - Add decorator for message headers #208

Triggered via pull request August 6, 2024 08:33
Status Failure
Total duration 6m 53s
Artifacts 2

build.yml

on: pull_request
Matrix: Matrix Build
Build and Test
0s
Build and Test
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 53 warnings
Matrix Build (3.3.1)
Gradle build failed: see console output for details
Matrix Build (3.3.1)
❌ No test results found for Test Report-(3.3.1)
Matrix Build (3.2.2)
❌ No test results found for Test Report-(3.2.2)
Matrix Build (3.2.2)
Gradle build failed: see console output for details
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/config/TkmsAutoConfiguration.java#L19
Wrong lexicographical order for 'java.util.Collections' import. Should be before 'org.springframework.context.annotation.Import'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/config/TkmsAutoConfiguration.java#L20
Wrong lexicographical order for 'java.util.List' import. Should be before 'org.springframework.context.annotation.Import'.
com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/api/TkmsMessage.java#L84
WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/TransactionalKafkaMessageSender.java#L150
Distance between variable 'transactionActive' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L15
Wrong lexicographical order for 'java.nio.charset.StandardCharsets' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L16
Wrong lexicographical order for 'java.util.Arrays' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L17
Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L18
Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L19
Wrong lexicographical order for 'java.util.concurrent.atomic.AtomicInteger' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L20
Wrong lexicographical order for 'java.util.stream.Stream' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L21
Wrong lexicographical order for 'java.util.stream.StreamSupport' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L23
Import statement for 'org.assertj.core.api.Assertions.assertThat' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L24
Import statement for 'org.awaitility.Awaitility.await' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L25
Import statement for 'org.junit.jupiter.api.Assertions.assertTrue' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/test/TestMessageDecorator.java#L7
Wrong lexicographical order for 'java.util.List' import. Should be before 'org.springframework.stereotype.Component'.
Matrix Build (3.3.1)
The following actions uses node12 which is deprecated and will be forced to run on node16: jwgmeligmeyling/checkstyle-github-action@master, jwgmeligmeyling/spotbugs-github-action@master. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Matrix Build (3.3.1)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, gradle/gradle-build-action@v2, mikepenz/action-junit-report@v3, jwgmeligmeyling/checkstyle-github-action@master, jwgmeligmeyling/spotbugs-github-action@master, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/config/TkmsAutoConfiguration.java#L19
Wrong lexicographical order for 'java.util.Collections' import. Should be before 'org.springframework.context.annotation.Import'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/config/TkmsAutoConfiguration.java#L20
Wrong lexicographical order for 'java.util.List' import. Should be before 'org.springframework.context.annotation.Import'.
com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/api/TkmsMessage.java#L84
WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: tw-tkms-starter/src/main/java/com/transferwise/kafka/tkms/TransactionalKafkaMessageSender.java#L150
Distance between variable 'transactionActive' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L15
Wrong lexicographical order for 'java.nio.charset.StandardCharsets' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L16
Wrong lexicographical order for 'java.util.Arrays' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L17
Wrong lexicographical order for 'java.util.HashMap' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L18
Wrong lexicographical order for 'java.util.Map' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L19
Wrong lexicographical order for 'java.util.concurrent.atomic.AtomicInteger' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L20
Wrong lexicographical order for 'java.util.stream.Stream' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L21
Wrong lexicographical order for 'java.util.stream.StreamSupport' import. Should be before 'org.springframework.beans.factory.annotation.Autowired'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L23
Import statement for 'org.assertj.core.api.Assertions.assertThat' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L24
Import statement for 'org.awaitility.Awaitility.await' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/MessageDecorationTest.java#L25
Import statement for 'org.junit.jupiter.api.Assertions.assertTrue' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: tw-tkms-starter/src/test/java/com/transferwise/kafka/tkms/test/TestMessageDecorator.java#L7
Wrong lexicographical order for 'java.util.List' import. Should be before 'org.springframework.stereotype.Component'.
Matrix Build (3.2.2)
The following actions uses node12 which is deprecated and will be forced to run on node16: jwgmeligmeyling/checkstyle-github-action@master, jwgmeligmeyling/spotbugs-github-action@master. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Matrix Build (3.2.2)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, gradle/gradle-build-action@v2, mikepenz/action-junit-report@v3, jwgmeligmeyling/checkstyle-github-action@master, jwgmeligmeyling/spotbugs-github-action@master, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "all-test-reports-3.2.2", "all-test-reports-3.3.1". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Artifacts

Produced during runtime
Name Size
all-test-reports-3.2.2 Expired
12.3 KB
all-test-reports-3.3.1 Expired
12.3 KB