Skip to content

Fix env format in Dockerfile #234

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

Conversation

amarburg
Copy link
Collaborator

@amarburg amarburg commented Aug 9, 2024

Changes Made

Fixes three ENVs into the preferred "key=value" format.

Associated Issues

Testing

docker build --check . produces:

ci

Check complete, no warnings found.

desktop

Check complete, no warnings found.

desktop-nvidia

Check complete, no warnings found.

robot

Check complete, no warnings found.

@amarburg amarburg changed the title 233 bug docker build check complains about legacy env format Fix env format in Dockerfile Aug 9, 2024
@amarburg amarburg force-pushed the 233-bug-docker-build-check-complains-about-legacy-env-format branch from fbeda76 to 2edc41e Compare August 9, 2024 19:26
@evan-palmer
Copy link
Collaborator

Going to wait for #220 before I merge this one

@evan-palmer
Copy link
Collaborator

This should be good to merge once the CI is done

@evan-palmer
Copy link
Collaborator

Looks like it needs a rebase first

…/blue into 233-bug-docker-build-check-complains-about-legacy-env-format
@amarburg
Copy link
Collaborator Author

Looks like it needs a rebase first

Rebased, waiting for CI ...

@evan-palmer evan-palmer merged commit 368b25a into Robotic-Decision-Making-Lab:main Aug 19, 2024
7 checks passed
@evan-palmer
Copy link
Collaborator

@mergify backport jazzy iron humble

Copy link

mergify bot commented Aug 19, 2024

backport jazzy iron humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)
mergify bot pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)
mergify bot pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)
evan-palmer pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)

Co-authored-by: Aaron Marburg <[email protected]>
evan-palmer pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)

Co-authored-by: Aaron Marburg <[email protected]>
evan-palmer pushed a commit that referenced this pull request Aug 19, 2024
Co-authored-by: Aaron Marburg <[email protected]>
(cherry picked from commit 368b25a)

Co-authored-by: Aaron Marburg <[email protected]>
@amarburg amarburg deleted the 233-bug-docker-build-check-complains-about-legacy-env-format branch August 23, 2024 16: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.

[BUG]: docker build --check complains about legacy ENV format
2 participants