CI: checks on every PR, autodeploy from main - #19
Merged
Conversation
Lint, format, types and the full vitest run (browser project included, so chromium gets installed) gate every pull request. A push to main that clears them rsyncs the static build to the box through the existing make target — the deploy key and host live in repo secrets. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BoX3XVqTj1NmubiDkHmWaQ
The old line covered code comments alone, which left commit messages, PR text and docs to guesswork — and one PR description already went out in Russian. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BoX3XVqTj1NmubiDkHmWaQ
freethinkel
force-pushed
the
fmc(pipelines)
branch
from
July 31, 2026 15:45
74ba8ac to
196a304
Compare
Reviewing a PR means running its postinstall scripts and its lint plugins, and the host has ssh keys on it. The image is the plain typescript-node one; the only two lines with a story are the node_modules volume, so a reviewed branch installs over its own copy rather than the host's, and the chown that makes that volume writable by the time the first install runs. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BoX3XVqTj1NmubiDkHmWaQ
Publishing the port made the second container fight the first one for it, and Zed was left with two candidates it couldn't choose between. Zed doesn't honour forwardPorts yet anyway — it tunnels a listening dev server on its own. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BoX3XVqTj1NmubiDkHmWaQ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One workflow, two jobs.
check (every PR + push to main):
oxlint→oxfmt --check→svelte-check→vitest(the browser project too, hence the chromium install). Green locally: 151 tests, 0 type errors.deploy (
needs: check, main only — so a merged PR counts): private key from the secret,ssh-keyscanof the host, then the existingmake deploy(pnpm build+ rsync into/root/fmc_pocketbase/www).DEPLOY_KEY/DEPLOY_HOSTsecrets are already set, the public key is in the server'sauthorized_keys, and the login was verified.🤖 Generated with Claude Code
https://claude.ai/code/session_01BoX3XVqTj1NmubiDkHmWaQ