Skip to content

Fix formatting in README for 01-setup-and-first-steps#87

Open
sebastiantecsi wants to merge 3 commits intogithub:mainfrom
sebastiantecsi:main
Open

Fix formatting in README for 01-setup-and-first-steps#87
sebastiantecsi wants to merge 3 commits intogithub:mainfrom
sebastiantecsi:main

Conversation

@sebastiantecsi
Copy link
Copy Markdown

@sebastiantecsi sebastiantecsi commented Apr 17, 2026

  • Fix formatting of allow-all command in README
  • Clarify instruction for summarizing utils.py

Copilot AI review requested due to automatic review settings April 17, 2026 22:35
Copy link
Copy Markdown
Contributor

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

This PR updates the permissions command table in the getting-started README to correct the formatting of the /allow-all command entry.

Changes:

  • Adjusted the /allow-all command’s argument list formatting within the Markdown table.

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

|---------|--------------|
| `/add-dir <directory>` | Add a directory to allowed list |
| `/allow-all [on|off|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status |
| `/allow-all [on\|off\|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status |
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

The backslashes in /allow-all [on\|off\|show] are inside an inline code span, so they will render literally (showing \|) rather than escaping the table delimiter. Consider reverting to /allow-all [on|off|show] inside backticks (GFM tables handle | inside code spans), or use an HTML entity for the pipe if you need to support parsers that don’t.

Suggested change
| `/allow-all [on\|off\|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status |
| `/allow-all [on|off|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status |

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

In fact \| is escaping the table delimiter, and that's what we need. Otherwise /allow-all [on|off|show] renders multiple | aka table delimiters.

cc @copilot

@sebastiantecsi sebastiantecsi changed the title Fix formatting of allow-all command in README Fix formatting in README for 01-setup-and-first-steps Apr 17, 2026
@sebastiantecsi sebastiantecsi requested a review from Copilot April 17, 2026 23:06
Copy link
Copy Markdown
Contributor

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 1 out of 1 changed files in this pull request and generated 1 comment.


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

Comment thread 01-setup-and-first-steps/README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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