Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggest using the new PHP feature attributes #84

Open
ezsky opened this issue Dec 9, 2023 · 2 comments
Open

Suggest using the new PHP feature attributes #84

ezsky opened this issue Dec 9, 2023 · 2 comments
Labels
status:ready for adoption Feel free to implement this issue. type:feature New feature

Comments

@ezsky
Copy link

ezsky commented Dec 9, 2023

Annotations are very frustrating to use. The IDE has poor support for it and is difficult to debug.
Suggest using the new PHP feature attributes,like https://github.com/uderline/openapi-php-attributes

@xepozz
Copy link
Member

xepozz commented Dec 11, 2023

Totally agree

@samdark samdark added status:ready for adoption Feel free to implement this issue. type:feature New feature labels Dec 13, 2023
@nkondrashov
Copy link

Hi all! doctrine/annotations being optional as of version 4.8 for zircote/swagger-php (current used ^4.0). Current annotation can't be parsed for openapi.json anymore (Current state after installation: demo apps have error on /docs, also yii-debug-api cant join openapi.js).

I suggest next scenario:

  1. Include doctrine/annotations in current composer json right now (for stabilization demo apps and yii-debug-api)
  2. Issuing refactoring demo apps and yii-debug-api to use native PHP annotation
  3. When complete step 2 - remove doctrine/annotations from composer json

p.s.
I hope I selected correctly place for this message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:ready for adoption Feel free to implement this issue. type:feature New feature
Projects
None yet
Development

No branches or pull requests

4 participants