Skip to content
This repository has been archived by the owner on Mar 20, 2023. It is now read-only.

rewrite conf file substitution processing #421

Merged
merged 7 commits into from
Jan 21, 2021
Merged

Conversation

kkaarreell
Copy link
Contributor

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kkaarreell, ycao56
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found 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

@sonarcloud
Copy link

sonarcloud bot commented Jan 21, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@kkaarreell
Copy link
Contributor Author

Hi @ChunxiAlexLuo , there is some failure in Travis but unrelated to cypress tests. Can you fix the problem or forcibly merge this PR? As the change is quite big I do not want to keep it open for a long time as I am afraid of conflicts with other PRs.

@ycao56 ycao56 merged commit 7bea008 into master Jan 21, 2021
@ycao56 ycao56 deleted the violation_parameters branch January 21, 2021 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cypress] add namespace parameter to config/violations-config.yaml
3 participants