Push/PR pipeline

Upgrade integrations SDK (NR-302384) (#175) #159

Sign in to view logs

Triggered via push October 4, 2024 08:32

kang-makes

master

Status Success

Total duration 2m 47s

Artifacts 1

on_push_pr.yaml

on: push

push-pr / ... / Test binary compilation for all platforms:arch

push-pr / ... / Run all static analysis checks

push-pr / ... / Run unit tests on *Nix

push-pr / ... / Run unit tests on Windows

push-pr / ... / Run integration tests on *Nix

push-pr / ... / Test binary compilation for all platforms:arch

Matrix: push-pr / test-build-fake-prerelease / Create MSI

Annotations

10 errors and 1 warning

push-pr / static-analysis / Run all static analysis checks: src/mssql.go#L92

exitAfterDefer: os.Exit will exit, and `defer con.Close()` will not run (gocritic)

push-pr / static-analysis / Run all static analysis checks: src/metrics/database_metric_definitions.go#L17

wrapperFunc: use strings.ReplaceAll method in `strings.Replace(query, databasePlaceHolder, dbName, -1)` (gocritic)

push-pr / static-analysis / Run all static analysis checks: src/metrics/metric_definitions_test.go#L28

wrapperFunc: use strings.ReplaceAll method in `strings.Replace(query, "%REPLACE%", "*", -1)` (gocritic)

push-pr / static-analysis / Run all static analysis checks: src/args/argument_list.go#L33

cyclomatic complexity 12 of func `(ArgumentList).Validate` is high (> 10) (gocyclo)

push-pr / static-analysis / Run all static analysis checks: src/metrics/metrics.go#L152

cyclomatic complexity 13 of func `populateCustomMetrics` is high (> 10) (gocyclo)

push-pr / static-analysis / Run all static analysis checks: src/connection/sql_connection.go#L55

unnecessary leading newline (whitespace)

push-pr / static-analysis / Run all static analysis checks: src/connection/sql_connection.go#L87

unnecessary trailing newline (whitespace)

push-pr / static-analysis / Run all static analysis checks: src/metrics/database_metric_definitions_test.go#L8

unnecessary leading newline (whitespace)

push-pr / static-analysis / Run all static analysis checks: src/args/argument_list.go#L36

do not define dynamic errors, use wrapped static errors instead: "errors.New(\"invalid configuration: must specify a hostname\")" (err113)

push-pr / static-analysis / Run all static analysis checks: src/args/argument_list.go#L40

do not define dynamic errors, use wrapped static errors instead: "errors.New(\"invalid configuration: specify either port or instance but not both\")" (err113)

push-pr / static-analysis / Run all static analysis checks

The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Artifacts

Produced during runtime

Name	Size
windows-packages	5 MB