Skip to content

Add repository READMEs, license, and contributor scaffolding#10

Merged
ianpittwood merged 9 commits into
mainfrom
readme-updates
May 5, 2026
Merged

Add repository READMEs, license, and contributor scaffolding#10
ianpittwood merged 9 commits into
mainfrom
readme-updates

Conversation

@ianpittwood
Copy link
Copy Markdown
Contributor

Summary

  • Replace the placeholder root README with a full overview of the specialized Workbench images, including a build/bakery workflow that mirrors images-workbench.
  • Add per-image READMEs under workbench-for-google-cloud-workstations/ and workbench-for-microsoft-azure-ml/, naming the canonical registry for each (GAR for Google Cloud Workstations, GHCR for Azure ML — with an explicit "do not use the MCR/ACR copy" callout).
  • Fix the repository.url in bakery.yaml (was pointing at images-workbench).
  • Add LICENSE.md (MIT, copy from images-workbench) and CODE_OF_CONDUCT.md (Contributor Covenant 2.1).
  • Add a justfile with setup, install-bakery, install-goss, and init recipes that match the contributor flow used in images-workbench.
  • Apply Posit doc-reviewer style fixes across all three READMEs (em dashes, possessives, passive voice, bold link text, may/can, slash in MCR/ACR, OCI definition).

Test plan

  • Render each README on GitHub and confirm tables, links, and the GitHub [!IMPORTANT] callout display correctly.
  • Confirm the four GAR registry rows in workbench-for-google-cloud-workstations/README.md resolve.
  • Verify just setup, just install-bakery, and just install-goss succeed on a clean checkout.
  • Confirm bakery build --plan still parses with the corrected repository.url.

🤖 Generated with Claude Code

ianpittwood and others added 4 commits May 4, 2026 15:00
Replace the placeholder root README with an overview of the specialized
Workbench images, and add per-image READMEs that point users at the
canonical registry (GAR for Google Cloud Workstations, GHCR for Azure
ML) and the platform-specific setup documentation. Also corrects the
repository URL in bakery.yaml.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Match images-workbench by including the project's MIT license and the
Contributor Covenant 2.1 code of conduct at the repository root.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Provide just recipes for installing pre-commit hooks, the bakery CLI,
and goss/dgoss, matching the contributor workflow used in
images-workbench.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Run the doc-reviewer skill against the new READMEs and address every
finding: replace em dashes with colons or sentence breaks, drop
possessives on company and product names (Google, Google Cloud,
Microsoft, Azure ML), convert passive constructions to active voice,
remove bold from non-UI text, replace "may" with "can" for possibility,
swap the slash in MCR/ACR for "and", and define OCI on first use.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ianpittwood ianpittwood requested a review from Copilot May 5, 2026 16:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CODE_OF_CONDUCT.md

Instances of abusive, harassing, or otherwise unacceptable behavior may be
reported to the community leaders responsible for enforcement at
[INSERT CONTACT METHOD].
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md
Comment thread justfile
Comment thread justfile
ianpittwood and others added 2 commits May 5, 2026 12:47
The repository builds with any OCI build tool (Docker, Podman, or
Buildah), so the root README should not single out Docker as the
prerequisite or in the no-local-tooling caveat.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The example code block already shows a podman build invocation; add
podman to the bulleted tool list above so the prose matches.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ianpittwood ianpittwood marked this pull request as ready for review May 5, 2026 19:10
@ianpittwood ianpittwood requested a review from bschwedler as a code owner May 5, 2026 19:10
ianpittwood and others added 3 commits May 5, 2026 13:49
Clarify that these images require product-specific licenses and that
standard Workbench licenses do not work. Link to the relevant Posit
solutions pages for licensing inquiries.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Add a pointer in the WGCW Image registries section to the US Central
Artifact Registry URL, which renders a tag listing in a browser.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ianpittwood ianpittwood merged commit b384231 into main May 5, 2026
5 of 6 checks passed
@ianpittwood ianpittwood deleted the readme-updates branch May 5, 2026 20:54
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