Skip to content

Fix Issue #1: DocBlock Errors#2

Open
Sental wants to merge 5 commits into
mage-os-lab:mainfrom
Sental:main
Open

Fix Issue #1: DocBlock Errors#2
Sental wants to merge 5 commits into
mage-os-lab:mainfrom
Sental:main

Conversation

@Sental
Copy link
Copy Markdown

@Sental Sental commented May 17, 2026

Summary

Motivation

Closes #1

How to test

  1. Run bin/magento setup:di:compile

Checklist

  • Branch is based on the latest main.
  • Commits follow Conventional Commits (feat:, fix:, refactor:, test:, chore:, docs:).
  • vendor/bin/phpunit --testsuite unit passes locally.
  • vendor/bin/phpstan analyse --memory-limit=1G passes locally.
  • vendor/bin/phpcs --standard=phpcs.xml.dist passes locally.
  • vendor/bin/php-cs-fixer fix --dry-run --diff --allow-risky=yes shows no changes needed.
  • New PHP files start with declare(strict_types=1);.
  • User-facing change has a CHANGELOG.md entry under ## [Unreleased].
  • No raw integer IDs in admin UX (use pickers / linked names. See CONTRIBUTING.md).

Screenshots / GraphQL samples (if UI or API change)

Sental and others added 5 commits May 17, 2026 14:12
Add DocBlocks to AuthorInterface
Add DocBlocks to the ExtensionAttributes Functions
Add DocBlocks to Extension Attributes Functions
Add DocBlocks to Extension Attributes functions
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.

Docblock Error during Setup:di:compile

1 participant