Skip to content

Network guides: Security row; CLAUDE.md pull-request rules restored - #57

Merged
robdevops merged 3 commits into
mainfrom
claude/kind-newton-ad0z2q
Oct 21, 2026
Merged

robdevops merged 3 commits into
mainfrom
claude/kind-newton-ad0z2q

Conversation

@robdevops

Copy link
Copy Markdown
Owner

What this PR holds

Every commit on the work branch since #56:

  • Network guides: the pull/push tables' "How finbot knows it is …" row is called Security in doc/slack.md, doc/telegram.md and doc/discord.md (dd23646).
  • CLAUDE.md: the "Pull requests from the work branch" rules and the PR sentence in the Iterating step are back, as a revert of 24742c3 (0b259ed).
  • Merge of main: one conflict, in the Slack table row. main had dropped the Verification Token (slackOutgoingToken) from it; I kept that, kept the row's new Security label, and tidied the sentence the edit left dangling ("…(slackSigningSecret). Without it, finbot turns Slack off").

Docs only, no code change: CI skips the tests for it.

Test plan

  • .venv/bin/python -m unittest discover -s tests -b (651 tests, offline) after the merge
  • CI

Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…PR and CI controls apply"

This reverts commit 24742c3, at the owner's request: the pull-request rules are back.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
@robdevops
robdevops merged commit f43aad0 into main Oct 21, 2026
4 checks passed
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