-
Notifications
You must be signed in to change notification settings - Fork 37
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
[NFR] Documentar su clase, hints para los IDE's #16
Comments
@gtux te refieres a esto https://www.phpdoc.org/ |
@marti1125 Sí, pero específicamente al @property-read, que es útil sobretodo en clases que tienen propiedades dinámicas, en este caso Culqi no tiene como atributos las clases Cargos o Suscripciones, sino que se establecen en el constructor, por lo que los IDE's no tienen conocimiento de esta variable y menos de su tipo, así que no te lo proponen al momento de escribir el código. Al documentar la clase con @property-read el IDE asume que de alguna manera existe ese atributo y es de tal tipo así que te lo ofrece en el autocompletado de código. |
@gtux listo lo reviso mañana 👍 algun otro feedback que tuvieras? |
@gtux hola!! aun lo sigues necesitando.... =/ aun el la lib para la version de la nueva api se sigue modificando |
No, no esperé a que hicieran la actualización, yo mismo lo arreglé en mi copia. |
@gtux la version 1.3.* de culqi-php ya tiene docs |
Los que usamos su librería oficial estaríamos agradecidos si ayudan al IDE para que genere automáticamente el descubrimiento de las clases y métodos internos, una forma podría ser agregar lo que el estándar PHPDoc menciona, y así tendríamos:
The text was updated successfully, but these errors were encountered: