CDCgov / trusted-intermediary

Bringing together healthcare providers by reducing the connection burden.
Apache License 2.0
11 stars 5 forks source link

Adding more context to readme #1605

Closed basiliskus closed 3 hours ago

basiliskus commented 3 days ago

Description

Issue

1540

github-actions[bot] commented 3 days ago

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis 🔶** **[1540](https://github.com/CDCgov/trusted-intermediary/issues/1540) - Partially compliant** Fully compliant requirements: - Instructions on how to test new assertions from a new branch/PR Not compliant requirements: - Checking for internal open/close quotes in json - Making tests fail on purpose as a gut check that they're running as expected/not passing because they're being filtered out
⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Missing Information
The PR does not address all the requirements specified in the ticket, particularly checking for quotes in JSON and making tests fail intentionally.
github-actions[bot] commented 3 days ago

PR Code Suggestions ✨

No code suggestions found for the PR.

sonarcloud[bot] commented 5 hours ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud