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

feat: add workdir to dockerfile #3259

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

smuu
Copy link
Member

@smuu smuu commented Mar 18, 2024

This should be a non-breaking change.

This only changes the work directory when the container is started. This means that if you want to execute a command in a running container, it will be automatically in that directory instead of '/'. This improves maintaining and debugging our environments.

Signed-off-by: Smuu <18609909+Smuu@users.noreply.github.com>
@github-actions github-actions bot added the external Issues created by non node team members label Mar 18, 2024
Copy link
Contributor

@tty47 tty47 left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.91%. Comparing base (2469e7a) to head (e0c9c6e).
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3259      +/-   ##
==========================================
+ Coverage   44.83%   44.91%   +0.08%     
==========================================
  Files         265      263       -2     
  Lines       14620    14634      +14     
==========================================
+ Hits         6555     6573      +18     
+ Misses       7313     7311       -2     
+ Partials      752      750       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@sysrex sysrex left a comment

Choose a reason for hiding this comment

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

lgtm

@ramin ramin added the kind:ci CI related PRs label Mar 19, 2024
@Wondertan Wondertan merged commit 0c5ed2d into celestiaorg:main Mar 19, 2024
26 of 27 checks passed
renaynay pushed a commit to renaynay/celestia-node that referenced this pull request Apr 3, 2024
This should be a non-breaking change.

This only changes the work directory when the container is started. This means that if you want to execute a command in a running container, it will be automatically in that directory instead of '/'. This improves maintaining and debugging our environments.
renaynay pushed a commit to renaynay/celestia-node that referenced this pull request Apr 3, 2024
This should be a non-breaking change.

This only changes the work directory when the container is started. This means that if you want to execute a command in a running container, it will be automatically in that directory instead of '/'. This improves maintaining and debugging our environments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external Issues created by non node team members kind:ci CI related PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants