Skip to content

Document demo branch context#81

Closed
dbpolito wants to merge 3 commits intomainfrom
docs/add-demo-md
Closed

Document demo branch context#81
dbpolito wants to merge 3 commits intomainfrom
docs/add-demo-md

Conversation

@dbpolito
Copy link
Copy Markdown
Contributor

@dbpolito dbpolito commented Apr 2, 2026

Reply to addressed PR review comments after removing demo-only comments from packages/opencode/index.ts.

@dbpolito dbpolito self-assigned this Apr 2, 2026
Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★★★☆

Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★★☆☆

Second instance of unhelpful comment added after prior review flagged the same issue on line 272.

Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★☆☆☆

DEMO.md contains placeholder text that should not be merged into the main branch. The content provides no documentation value. If this PR is meant to document the demo branch, DEMO.md should contain actual useful documentation. If DEMO.md was created for demo purposes only (like the removed index.ts comments), it should be removed from this PR before merging.

@@ -0,0 +1 @@
This is a file i'm using in the demo. GOOD STUFF.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This placeholder content ("This is a file i'm using in the demo. GOOD STUFF.") should not be merged into main. Either:

  1. Remove DEMO.md entirely (if it's demo-only content), or
  2. Replace with actual documentation about the demo branch

The prior reviews flagged demo-only comments in packages/opencode/index.ts, which were correctly removed. DEMO.md appears to be the same category of demo-only content.

@dbpolito dbpolito closed this Apr 2, 2026
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