Skip to content

fix(readme): use literal quoting for marketplace URL#108

Merged
kantorcodes merged 2 commits into
mainfrom
fix/codex-marketplace-install-url-single-quotes
May 15, 2026
Merged

fix(readme): use literal quoting for marketplace URL#108
kantorcodes merged 2 commits into
mainfrom
fix/codex-marketplace-install-url-single-quotes

Conversation

@kantorcodes
Copy link
Copy Markdown
Member

Summary:

  • use single quotes around Codex marketplace Git repo URL
  • addresses Gemini review from PR 107 after PR auto-merged

Verification:

  • README install command check passed
  • marketplace JSON validates with jq
  • git diff --check passed
  • README guidance guard script passed

Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 15, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • README.md - No issues

Reviewed by step-3.5-flash · 69,153 tokens

Copy link
Copy Markdown
Contributor

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

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 updates the README.md file to use single quotes for a URL in a shell command example. The reviewer suggests extending this literal quoting to all other arguments in the command to ensure consistency and prevent unintended shell expansion.

Comment thread README.md Outdated
Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
@kantorcodes kantorcodes merged commit a89a390 into main May 15, 2026
9 checks passed
@kantorcodes kantorcodes deleted the fix/codex-marketplace-install-url-single-quotes branch May 15, 2026 20:00
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.

1 participant