Skip to content

Fix incorrect and inconsistent docstrings across resources#207

Open
JoshSalway wants to merge 1 commit into
laravel:3.xfrom
JoshSalway:fix/docstring-corrections
Open

Fix incorrect and inconsistent docstrings across resources#207
JoshSalway wants to merge 1 commit into
laravel:3.xfrom
JoshSalway:fix/docstring-corrections

Conversation

@JoshSalway
Copy link
Copy Markdown

@JoshSalway JoshSalway commented Mar 17, 2026

Fixes #208

Summary

Corrects copy-paste errors and incorrect docstrings across 5 files:

  • SecurityRule.php: $credentials and delete() docstrings incorrectly say "redirect rule" — fixed to "security rule"
  • SiteCommand.php: $updatedAt docstring has duplicate "The The" — removed duplicate
  • Site.php: $tags property docstring says "aliases" — fixed to "tags"
  • BackupConfiguration.php: $backups property docstring says "databases" — fixed to "backups"
  • ManagesSiteCommands.php: listCommandHistory() return type incorrectly documented as SiteCommand — fixed to SiteCommand[] since the method returns an array

Test plan

  • Docstring-only changes — no runtime behaviour affected
  • listCommandHistory() return type fix improves IDE autocompletion accuracy

🤖 Generated with Claude Code

…d action files

- Fix "redirect rule" -> "security rule" in SecurityRule (fixes #8)
- Remove duplicate "The The" in SiteCommand docstring (fixes #9)
- Fix "aliases" -> "tags" in Site::$tags docstring (fixes #10)
- Fix "databases" -> "backups" in BackupConfiguration::$backups docstring (fixes #11)
- Fix return type on listCommandHistory() to SiteCommand[] (fixes #12)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Incorrect and inconsistent docstrings across resource and action files

1 participant