search

RMI-PACTA / workflow.pacta

Other
2 stars 2 forks source link

Set `USER` earlier in Dockerfile #72

Closed AlexAxthelm closed 4 months ago

AlexAxthelm commented 5 months ago

Create the workflow-pacta user immediately after installing system dependencies and creating the protected Rprofile.site, and add that user to staff group, giving permissions to install R packages to the site library.

No assosciated issue, reflecting practice in https://github.com/RMI-PACTA/workflow.pacta.report/pull/14

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@c3376be). Learn more about missing BASE report.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #72 +/- ## ====================================== Coverage ? 0.00% ====================================== Files ? 4 Lines ? 256 Branches ? 0 ====================================== Hits ? 0 Misses ? 256 Partials ? 0 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 5 months ago

Docker build status

commit_time git_sha image
2024-05-28T12:59:57Z 9058ca84dc5a7a39bb9d9c9d1e18845f53f57da8 ghcr.io/rmi-pacta/workflow.pacta:pr-72
AlexAxthelm commented 5 months ago

Converting to draft, until I sort out the UNIX permissions issues disvocered in https://github.com/RMI-PACTA/workflow.pacta.report/pull/14

AlexAxthelm commented 4 months ago

Closing in favor of a bigger docker refactoring to come later.