Skip to content

Updated Dockerfile and switch away from bitnami#213

Open
cableman wants to merge 2 commits intoOS2iot:stagefrom
itk-dev:feature/docker-setup
Open

Updated Dockerfile and switch away from bitnami#213
cableman wants to merge 2 commits intoOS2iot:stagefrom
itk-dev:feature/docker-setup

Conversation

@cableman
Copy link
Copy Markdown

@cableman cableman commented Jan 13, 2026

See OS2iot/OS2iot-docker#23 this PR update the nginx image

@fcv-iteratorIt fcv-iteratorIt changed the base branch from master to stage February 27, 2026 06:59
Copy link
Copy Markdown
Contributor

@fcv-iteratorIt fcv-iteratorIt left a comment

Choose a reason for hiding this comment

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

Getting following warnings that might as well be updated:

- FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 3)
- LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 15)

@cableman
Copy link
Copy Markdown
Author

cableman commented Mar 9, 2026

Fixed the warnings (what tool did you use to verify this?) and it was outside the scope of this PR.

@cableman cableman requested a review from fcv-iteratorIt March 9, 2026 13:47
@rimi-itk
Copy link
Copy Markdown

Getting following warnings that might as well be updated:

- FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 3)
- LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 15)

@cableman, these are reported when running docker build --check ..

@rimi-itk
Copy link
Copy Markdown

@cableman @fcv-iteratorIt itk-dev#1 cleans up a little more.

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.

3 participants