Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filter out the .tekton and vendor from shellcheck #40

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

xinredhat
Copy link
Member

@xinredhat xinredhat commented Jun 21, 2024

This PR depends on another PR konflux-ci/tekton-integration-catalog#17

@xinredhat xinredhat changed the title [WIP]filter out the .tekton and vendor from shellcheck filter out the .tekton and vendor from shellcheck Jun 21, 2024
@xinredhat
Copy link
Member Author

"test" directory is filtered out in the PR #39 , which should fix the failure that yamllint reported in CI Red Hat Konflux / rhtap-cli-static-code-analysis / rhtap-cli

@konflux-ci-qe-bot
Copy link

@xinredhat: The following test finished, in case test failed say /retest to rerun all failed tests:

PipelineRun name Command
rhtap-cli-vnrfx-hrpx7 /retest

To inspect your test artifacts make sure you have installed ORAS in your local machine. To inspect all test artifacts execute:

mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/konflux-qe-incubator/konflux-qe-oci-storage:rhtap-cli-vnrfx-hrpx7

For instructions on how to install ORAS, please refer to the ORAS installation guide.

@konflux-ci-qe-bot
Copy link

@xinredhat: The following test finished, in case test failed say /retest to rerun all failed tests:

PipelineRun name Command
rhtap-cli-q84gk-wrwfq /retest

To inspect your test artifacts make sure you have installed ORAS in your local machine. To inspect all test artifacts execute:

mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/konflux-qe-incubator/konflux-qe-oci-storage:rhtap-cli-q84gk-wrwfq

For instructions on how to install ORAS, please refer to the ORAS installation guide.

@Roming22 Roming22 merged commit 2b1063a into redhat-appstudio:main Jun 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants