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

fix recursive variable reference resolution #677

Merged

Conversation

patilpankaj212
Copy link
Contributor

No description provided.

@patilpankaj212 patilpankaj212 linked an issue Apr 19, 2021 that may be closed by this pull request
@sonarcloud
Copy link

sonarcloud bot commented Apr 19, 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

@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #677 (3b6a7c9) into master (2033718) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #677   +/-   ##
=======================================
  Coverage   74.97%   74.97%           
=======================================
  Files         110      110           
  Lines        3061     3061           
=======================================
  Hits         2295     2295           
  Misses        592      592           
  Partials      174      174           
Impacted Files Coverage Δ
...providers/terraform/commons/variable-references.go 81.96% <100.00%> (ø)
pkg/results/types.go 100.00% <0.00%> (ø)
pkg/runtime/rules.go 100.00% <0.00%> (ø)
pkg/policy/types.go 100.00% <0.00%> (ø)
pkg/iac-providers/kustomize/v3/load-file.go 100.00% <0.00%> (ø)
pkg/policy/gcp.go 100.00% <0.00%> (ø)
pkg/http-server/handler.go 100.00% <0.00%> (ø)
pkg/notifications/register.go 100.00% <0.00%> (ø)
pkg/writer/yaml.go 100.00% <0.00%> (ø)
pkg/logging/logger.go 100.00% <0.00%> (ø)
... and 101 more

@kanchwala-yusuf kanchwala-yusuf merged commit 585edcc into tenable:master Apr 21, 2021
@patilpankaj212 patilpankaj212 linked an issue Apr 22, 2021 that may be closed by this pull request
@patilpankaj212 patilpankaj212 mentioned this pull request May 5, 2021
@patilpankaj212 patilpankaj212 deleted the recursive-var-resolve-fix branch May 5, 2022 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants