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

terrascan argo-cd instructions #454

Merged
6 commits merged into from
Jan 14, 2021
Merged

terrascan argo-cd instructions #454

6 commits merged into from
Jan 14, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jan 6, 2021

No description provided.

@ghost ghost requested review from cesar-rodriguez and acc-jon January 6, 2021 03:11
@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #454 (cf4e248) into master (5c565e6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #454   +/-   ##
=======================================
  Coverage   74.69%   74.69%           
=======================================
  Files          88       88           
  Lines        2039     2039           
=======================================
  Hits         1523     1523           
  Misses        382      382           
  Partials      134      134           

cesar-rodriguez
cesar-rodriguez previously approved these changes Jan 6, 2021
docs/cicd.md Outdated Show resolved Hide resolved
docs/cicd.md Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Jan 13, 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
No Duplication information No Duplication information

Copy link
Contributor

@acc-jon acc-jon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@acc-jon acc-jon requested a review from jlk January 13, 2021 01:07
Copy link
Contributor

@jlk jlk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Looks like the right tag's there but not shown in the PR somehow??

@ghost ghost merged commit 5a19951 into master Jan 14, 2021
@ghost ghost deleted the argo-cd-hook branch January 14, 2021 00:20
This pull request was closed.
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.

4 participants