Skip to content

feat(docs): add github app deployment option - #298

Merged
mdong1909 merged 2 commits into
mainfrom
docs-git-deploy
Aug 28, 2026
Merged

mdong1909 merged 2 commits into
mainfrom
docs-git-deploy

Conversation

@mdong1909

Copy link
Copy Markdown
Contributor

Documents github app deployment

Comment thread docs/guide/deployment.md Outdated
void github join
```

::: 3. Link your repository to your project

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Markdown syntax error: The closing ::: tag for the details block is incorrectly combined with step 3's header on the same line. This will cause the details block to not close properly and step 3 to be malformed.

The line should be split:

:::

3. Link your repository to your project
Suggested change
::: 3. Link your repository to your project
:::
3. Link your repository to your project

Spotted by Graphite

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a6934b73bb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/guide/deployment.md Outdated
void github join
```

::: 3. Link your repository to your project

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Close the details container before step 3

The closing marker must be ::: on its own line. With 3. Link your repository... appended to it, VitePress does not close the details container, so steps 3–6 and the subsequent GitHub Actions and Other Targets sections are rendered inside the collapsed “someone already installed the App” block.

Useful? React with 👍 / 👎.

Comment thread docs/guide/deployment.md
6. Follow the build

```bash
void build logs --follow

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Wait for the new build before following logs

When the GitHub webhook has not created the new build by the time this command runs, runBuildLogsCommand selects the newest existing build only once (packages/void/src/cli/build-cmd.ts, lines 123–132). Following these consecutive steps can therefore show a previous completed build—or exit with “No builds found”—rather than follow the push from step 5; the instructions should wait until the new build appears or identify that build explicitly.

Useful? React with 👍 / 👎.

@mdong1909
mdong1909 merged commit 9d47ac7 into main Aug 28, 2026
8 checks passed
@mdong1909
mdong1909 deleted the docs-git-deploy branch August 28, 2026 08:03
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