Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Disable gosec G101 check, until proper version is used in golangci #746

Merged
merged 2 commits into from
Oct 22, 2023

Conversation

skabashnyuk
Copy link
Collaborator

What does this PR do?

Disable gosec G101 check, until the proper version is used in golangci

Screenshot/screencast of this PR

Знімок екрана 2023-10-22 о 15 10 52

What issues does this PR fix or reference?

securego/gosec#1046

How to test this PR?

golangci-lint run

@openshift-ci
Copy link

openshift-ci bot commented Oct 22, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: skabashnyuk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link

openshift-ci bot commented Oct 22, 2023

@skabashnyuk: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/spi-service-e2e f7346d9 link true /test spi-service-e2e

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@skabashnyuk skabashnyuk merged commit 3c3669c into konflux-ci:main Oct 22, 2023
9 of 10 checks passed
@skabashnyuk skabashnyuk deleted the disable-g101 branch October 25, 2023 06:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant