IIC2513-2021-1 / projects

Repositorio oficial para los proyectos de ambas secciones del periodo 2021-1 del curso PUC IIC2513
14 stars 6 forks source link

[E2.2] Error linter airbnb: uso de prop types #112

Open Baelfire18 opened 3 years ago

Baelfire18 commented 3 years ago

Hola seguimos los pasos de las capsulas y nos tira error el linter de airbnb debido a la forma de usar los props image image

Despues de buscar en internet, descrubirmos que hay que definir los proptypes, el problema de ello es que hay que usar una dependencia prop-types, la cosa es que como no se explica en ninguna capsula. No sabemos si es necesario hacerlo así o hay alguna mejor forma recomendada por el cuerpo docente.

Pudimos hacerlo funcionar ignorando este problema, es legal que hagamos eso? Siempre y cuando lo conversemos con nuestro ayudante y nos autorice claro.

Muchas gracias!

F-arellano commented 3 years ago

Tenía el mismo problema, y encontré la solución acá https://github.com/yannickcr/eslint-plugin-react/issues/2135 Simplemente cambié el nombre de la variable 'props' por cualquier otro (en mi caso usé prop xd). No sé si es lo recomendado, pero solucionas el error simplemente borrando 2 letras en el código

meretamal commented 3 years ago

Hola 😁 , si bien la solución correcta para ignorar el error es la que dice @Baelfire18, lo que recomendamos es que efectivamente hagan uso de prop-types, que les permitirá definir el tipo de los props que reciba un componente, con tal de que sea más consistente y hagan uso de buenas prácticas.

Este link de la documentación explica bien cómo usarlos. Es agregar un par de líneas sin mucha lógica y complejidad por detrás.

Si bien para esta entrega les dejamos ignorar esa regla de ESlint y no hacer uso de prop-types, es posible que sus ayudantes se los puedan pedir como requisito para la E2.3, tomando en cuenta el enfoque en la calidad del software que se está intentando implementar.