Skip to content

Recommend s7 as primary package in docs#667

Merged
gijzelaerr merged 1 commit into
masterfrom
docs-recommend-s7-package
Apr 3, 2026
Merged

Recommend s7 as primary package in docs#667
gijzelaerr merged 1 commit into
masterfrom
docs-recommend-s7-package

Conversation

@gijzelaerr
Copy link
Copy Markdown
Owner

Summary

  • Update all documentation and code examples to recommend s7 as the primary package for all PLC models (S7-300/400/1200/1500), with snap7 kept for backwards compatibility
  • Add s7.util module that re-exports all snap7.util helpers, so users never need to import snap7 directly
  • Update source docstrings in snap7 package to mark classes as legacy and point to s7 equivalents
  • Restructure API reference: s7.Client docs listed first, snap7.Client docs labeled as legacy

Changes

  • README.rst: Quick Start uses s7.Client, legacy example shown separately
  • doc/: All 12 prose pages updated (introduction, connecting, reading-writing, server, multi-variable, connection-issues, thread-safety, plc-support, limitations, etc.)
  • doc/API/: Restructured — s7commplus.rst is now "S7 Client (recommended)", client.rst/async_client.rst labeled "(legacy)", server.rst documents both
  • s7/util.py: New module re-exporting snap7.util.*
  • snap7/init.py, client.py, async_client.py, server/init.py: Docstrings updated to say "legacy" and point to s7

Test plan

  • Sphinx docs build with -W (warnings-as-errors) — no errors
  • ruff check passes on all changed files
  • mypy passes on s7/util.py

🤖 Generated with Claude Code

Update all documentation, examples, and source docstrings to present the
s7 package as the recommended entry point for all PLC models, with snap7
kept for backwards compatibility. Add s7.util module re-exporting
snap7.util so users never need to import snap7 directly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gijzelaerr gijzelaerr merged commit f2f5118 into master Apr 3, 2026
38 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.

1 participant