heimdall2
heimdall2 copied to clipboard
Azure policy converter
@Amndeep7 @camdenmoors @ejaronne -- Can you guys review this for me and give feedback? I know it's tough without access to Microsoft Azure. Please let me know if you'd like to hop on a Teams call and go through the code.
This pull request introduces 6 alerts when merging a133c55f823899ced127e93dba5dc61e312a7a1c into df946a8e13d5deb01e60c26e9f768d2dc59473b5 - view on LGTM.com
new alerts:
- 5 for Unused variable, import, function or class
- 1 for Useless conditional
@jrwhite17 please set up a meeting for the latter half of this week (I can be there) / beginning of next week (I will probably not be there due to being on leave). in the mean time, please address the failing tests from the automation.
Sounds good! I'll fix these issues then schedule something with you guys.
This pull request introduces 3 alerts when merging d93c2e696d9ecca655424085ad7154450362735d into f4eb1887a5d3a9395e11f97bde925273f419fc0a - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging a3bf5944b7c5b03bbd5830184cbc81d7214c861b into f4eb1887a5d3a9395e11f97bde925273f419fc0a - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging 7d8ab1ea3701f93945b266850ef216a7d3436cc4 into f4eb1887a5d3a9395e11f97bde925273f419fc0a - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging 360a323844d19a4a3a69e6ee20c48343f17f01a7 into f4eb1887a5d3a9395e11f97bde925273f419fc0a - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging 54547b9ae6680a42c52689346946e6bcde91dbe3 into 61e077d5773bdec588508c6227f6ec09d73970a9 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging 01fe9937ef833b7ccebf45f132b23156e4bd5b32 into 61e077d5773bdec588508c6227f6ec09d73970a9 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 4 alerts when merging 50c7a5393e608cda7e5c57cb2d5bd1e5e2d81550 into 81d251437f00afc6fe7646f154d2d637c9b8692c - view on LGTM.com
new alerts:
- 3 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging 2a67bef19acc29514cd424ce0b8c4a558bfa5ffe into 19f60a2830a39d8a1e61a85d38427b292d763fb2 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 3 alerts when merging a0cb406e5fd37ad27b961b2d1f071a62b5ce8555 into c4416ecbc0e4b1d6c96e1df031674f5221fd0222 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
SonarCloud Quality Gate failed. 
0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells
No Coverage information
0.0% Duplication
This pull request introduces 3 alerts when merging ffb54c9a62a0f734844c2fce4a1665aff1f50238 into a1d0e365906f6781e29ff615e48d174ebe766332 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request introduces 3 alerts when merging fcf60a3465eb65cecea80264f8d37dfba7293c4d into a7b53f4e29398a3e33e7a0440e790be5cb5a3b66 - view on LGTM.com
new alerts:
- 2 for Unused variable, import, function or class
- 1 for Useless conditional
This pull request introduces 2 alerts when merging e7a9fe1ea4bf459455766b47b90d42a3fe99c54c into a7b53f4e29398a3e33e7a0440e790be5cb5a3b66 - view on LGTM.com
new alerts:
- 1 for Useless conditional
- 1 for Unused variable, import, function or class
This pull request introduces 1 alert when merging ab34ee8a84261da08bc67bc95cd85ea91f58795d into a7b53f4e29398a3e33e7a0440e790be5cb5a3b66 - view on LGTM.com
new alerts:
- 1 for Useless conditional
I think if we retry the failing Cypress build step a couple times it'll probably go through. If it doesn't, let's sit down and figure out what's the underlying issue @Rlin232 @camdenmoors
Kudos, SonarCloud Quality Gate passed! 
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?
This pull request has a conflict. Could you fix it @jrwhite17?