Skip to content
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

create home dir in dev-tools container #2820

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

msnwatson
Copy link
Contributor

What was the problem?

When trying to do some adhoc analyses in our dev-tools container, I realized there was a need for some file manipulation that was not able to be performed.

Associated tickets or Slack threads:

How does this fix it?1

creates a home directory for file manipulation

How to test this PR

  • Deployed to dev

Footnotes

  1. Pull-Requests guidelines. If PR is significant, update Current Software State wiki page.

@msnwatson msnwatson requested a review from a team as a code owner April 3, 2024 18:33
Copy link
Contributor

github-actions bot commented Apr 3, 2024

Test Results

151 tests  ±0   151 ✅ ±0   47s ⏱️ -1s
 46 suites ±0     0 💤 ±0 
 46 files   ±0     0 ❌ ±0 

Results for commit 3aa7a09. ± Comparison against base commit 032c4ec.

Copy link
Contributor

github-actions bot commented Apr 3, 2024

JaCoCo Test Coverage

Overall Project 76.55%

There is no coverage information present for the Files changed

Copy link
Contributor

@nelsestu nelsestu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM especially since there are other possible enhancements that can build off of this very simple change.

@msnwatson msnwatson merged commit dcf30b5 into develop Apr 4, 2024
1 check passed
@msnwatson msnwatson deleted the mwatson/create-home-dir branch April 4, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants