Skip to content

Bring README and repository files up to the organization standard - #8

Merged
bonelifer merged 3 commits into
masterfrom
readme-compliance
Oct 28, 2026
Merged

bonelifer merged 3 commits into
masterfrom
readme-compliance

Conversation

@bonelifer

Copy link
Copy Markdown
Contributor

Brings the README and repository files up to the phpbbmodders standard.

  • README: rewritten in the organization layout: Tests and Lint badges, Features, Screenshots, Requirements, Installation, Upgrading from rmcgirr83/posteripinviewtopic (Rich McGirr's original), Contributing, Acknowledgments, License. The Features section notes that the lookup sends the poster's IP address to ip-api.com over plain HTTP, and Requirements lists PHP's cURL extension.
  • Screenshots: the IP address in a post's profile column, and the lookup popup. Taken with phpbb-test-board/screenshots.py on a seed-forum board (scenario added in phpbbmodders/phpbb-release-tools).
  • Lint workflow: php -l and composer validate on PHP 8.3 and 8.4.
  • .gitattributes: * text=auto eol=lf, binary rules for images, docs/images/ and the README excluded from release ZIPs (the old README.MD rule never matched README.md).
  • Line endings: .gitignore was stored with CRLF; normalized to LF in a separate commit.
  • Typo: "PPoster" in the language/en/common.php header.

No PHP or template code changed.


Investigated and written by Claude on behalf of William Jacoby (bonelifer).

bonelifer and others added 3 commits October 28, 2026 10:39
Rewrite the README in the organization layout (badges, features,
requirements, installation, upgrading from rmcgirr83, contributing,
acknowledgments, license), add the Lint workflow, and add LF and
binary rules to .gitattributes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Taken with phpbb-release-tools/phpbb-test-board/screenshots.py on a seed-forum board.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bonelifer
bonelifer merged commit f808f12 into master Oct 28, 2026
8 checks passed
@bonelifer
bonelifer deleted the readme-compliance branch October 28, 2026 15:48
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