Closed Jeff-Thompson12 closed 5 months ago
Attention: 116 lines
in your changes are missing coverage. Please review.
Comparison is base (
12e418d
) 78.12% compared to head (b6fb751
) 77.19%.
Files | Patch % | Lines |
---|---|---|
R/mod_user_roles.R | 1.09% | 90 Missing :warning: |
R/app_server.R | 33.33% | 6 Missing :warning: |
R/mod_decision_automation.R | 61.53% | 5 Missing :warning: |
R/mod_reportPreview.R | 40.00% | 3 Missing :warning: |
R/utils_config_db.R | 50.00% | 3 Missing :warning: |
R/utils_startup.R | 81.81% | 2 Missing :warning: |
R/mod_addComment.R | 75.00% | 1 Missing :warning: |
R/mod_communityMetrics.R | 50.00% | 1 Missing :warning: |
R/mod_introJS.R | 50.00% | 1 Missing :warning: |
R/mod_maintenanceMetrics.R | 50.00% | 1 Missing :warning: |
... and 3 more |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@Jeff-Thompson12, after recent merges, there is now a conflict on this branch.
I think this looks really good. The only thing I would advise is to test out creating roles within our Posit Connect environment before merging. That way, we can see if they behave as we expect them to.
Nice work!
Addresses #700