Skip to content

docs: rebase app README refresh#16

Closed
sedge-bot wants to merge 1 commit into
fielding:mainfrom
sedge-bot:sedge/app-readme-refresh-current
Closed

docs: rebase app README refresh#16
sedge-bot wants to merge 1 commit into
fielding:mainfrom
sedge-bot:sedge/app-readme-refresh-current

Conversation

@sedge-bot
Copy link
Copy Markdown

Summary

  • rebase the app README refresh onto current main
  • keep the PR scoped to packages/app/README.md only, avoiding the stale branch's large app-sol/pnpm-lock reverse-diff
  • document Node 22, root-level pnpm commands, key public env vars, and the main app surfaces

Supersedes #9, whose branch is stale against current main and triggers failing Vercel checks on unrelated old app state.

Verification

  • git diff --check origin/main..HEAD
  • corepack pnpm --filter app test (97 passed)
  • corepack pnpm --filter app build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

@sedge-bot is attempting to deploy a commit to the Fielding Johnston's projects Team on Vercel.

A member of the Team first needs to authorize it.

@sedge-bot
Copy link
Copy Markdown
Author

Heartbeat verification for this docs-only PR at a2ae02a:

  • Diff scope is still only packages/app/README.md.
  • corepack pnpm --filter app test passed: 97 tests.
  • corepack pnpm --filter app-sol test passed: 131 tests.
  • git diff --check origin/main..HEAD passed.
  • corepack pnpm --filter app lint and corepack pnpm --filter app-sol lint still fail on pre-existing React Compiler / eslint issues outside the README diff (Date.now() during render and synchronous setState in effect in app/app-sol components). I did not change code on this docs PR.

GitHub marks the PR UNSTABLE only because the three Vercel checks are authorization failures for sedge-bot, not because of a code/test failure surfaced by this verification run.

@sedge-bot
Copy link
Copy Markdown
Author

Superseded by #17.

I rechecked this branch after origin/main advanced and found PR #16 was no longer README-only: origin/main..HEAD included unrelated reverse-diff in create-flow/schedule files. I recreated the intended README refresh on current main as #17 (7243f17) and verified:

  • corepack pnpm --filter app test (106 passed)
  • corepack pnpm --filter app-sol test (140 passed)
  • git diff --check origin/main..HEAD

The remaining failing checks on #17 are the same Vercel fork-preview authorization failures, not local test failures.

@sedge-bot
Copy link
Copy Markdown
Author

Closed in favor of replacement PR #17, recreated on current main with only the intended README change.

@sedge-bot sedge-bot closed this May 15, 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