Skip to content

Update readme and dockerfile - #220

Merged
thelovekesh merged 3 commits into
mainfrom
update/readme
May 17, 2026
Merged

Update readme and dockerfile#220
thelovekesh merged 3 commits into
mainfrom
update/readme

Conversation

@thelovekesh

@thelovekesh thelovekesh commented May 17, 2026

Copy link
Copy Markdown
Collaborator
  • Add command in readme for installation using go.
  • Update dockerfile to have binary target to keep build clean.

@thelovekesh thelovekesh added documentation Improvements or additions to documentation enhancement New feature or request labels May 17, 2026
@github-actions

Copy link
Copy Markdown

📦 wpm cli builds 📦

@thelovekesh
thelovekesh merged commit 3c09aa2 into main May 17, 2026
17 checks passed
@thelovekesh
thelovekesh deleted the update/readme branch May 17, 2026 11:26

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new image stage in the Dockerfile, adds Go installation instructions to the README, and updates the Docker Bake configuration. Feedback suggests using a more standard PATH or binary location in the Dockerfile, adding language tags to README code blocks for better syntax highlighting, and maintaining consistent attribute ordering (placing inherits at the top) in the docker-bake.hcl file.

Comment thread Dockerfile
Comment thread README.md
Comment thread docker-bake.hcl
Comment thread docker-bake.hcl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant