-
Notifications
You must be signed in to change notification settings - Fork 31
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
Allow a non-credentialed deployment #703
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #703 +/- ##
==========================================
- Coverage 78.12% 77.19% -0.94%
==========================================
Files 33 33
Lines 4897 4964 +67
==========================================
+ Hits 3826 3832 +6
- Misses 1071 1132 +61 ☔ View full report in Codecov by Sentry. |
@Jeff-Thompson12, after recent merges, there is now a conflict on this branch. |
Merge branch 'dev' into jt-700-noncredential_deploy # Conflicts: # NEWS.md # R/utils_startup.R
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! |
Merge branch 'dev' into jt-700-noncredential_deploy # Conflicts: # NEWS.md
Merge branch 'dev' into jt-700-noncredential_deploy # Conflicts: # NEWS.md
Merge branch 'dev' into jt-700-noncredential_deploy # Conflicts: # NEWS.md # R/utils_startup.R
Merge branch 'dev' into jt-700-noncredential_deploy # Conflicts: # NEWS.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✔️
Addresses #700