Skip to content

docs: pull the release before running update.py - #24

Merged
Pankraz01 merged 1 commit into
mainfrom
docs/deploy-pull-before-update
Sep 14, 2026
Merged

Pankraz01 merged 1 commit into
mainfrom
docs/deploy-pull-before-update

Conversation

@Pankraz01

Copy link
Copy Markdown
Contributor

scripts/update.py rebuilds and restarts the checked-out tree; it never fetches commits (--pull only refreshes Docker base images). Section 14 of DEPLOY.md now shows the git pull --ff-only origin main step before update.py, which is what the 2026-09-13 deploy needed.

🤖 Generated with Claude Code

update.py does not fetch commits; its --pull flag only refreshes the Docker base images.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Pankraz01
Pankraz01 merged commit 0a445e8 into main Sep 14, 2026
1 check passed
@Pankraz01
Pankraz01 deleted the docs/deploy-pull-before-update branch September 14, 2026 20:51

@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: a1648a7e30

ℹ️ 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 DEPLOY.md

```bash
cd /srv/scamscreener
git pull --ff-only origin main

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep updates compatible with uploaded deployments

Section 7 instructs operators to upload repository contents and never requires /srv/scamscreener to be a Git checkout with an origin remote, so deployments created exactly as documented can fail here with “not a git repository.” When this multi-line block is pasted into an interactive shell, Bash still proceeds to update.py, rebuilding the stale tree and potentially reporting a successful update. Either make the initial installation explicitly clone/configure this repository or retain an upload-based update path and stop execution if release acquisition fails.

AGENTS.md reference: AGENTS.md:L121-L125

Useful? React with 👍 / 👎.

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