Skip to content

Traducción de archivo src/groups/create.js de JS a TS#41

Open
oliebueno wants to merge 4 commits intoUSB-CI3715:f24from
oliebueno:f24
Open

Traducción de archivo src/groups/create.js de JS a TS#41
oliebueno wants to merge 4 commits intoUSB-CI3715:f24from
oliebueno:f24

Conversation

@oliebueno
Copy link

En este Pull request resuelve el issues #20. Para resolverlo se hizo la traducción del arcvhivo src/groups/create.js de JS a TS. En el archivo se realizaron los siguientes cambios:

  • Se modificaron las sintaxis de require y export para que fueran acordes con las usadas en TypeScript (import - export)
  • Se implemetaron interface (GroupData, GroupDataComplete y GroupsInterface) para poder tipar correctamenta a las variables y a las funciones usadas.
  • En varias parte del archivo se agregaron los comentarios @typescript-eslint/no-unsafe-call y @typescript-eslint/no-unsafe-member-access para ignorar las advertencias del lint debido a que existen archivos que aún no han sido traducidos a TypeScript.
  • Se corrigió el uso de any.
  • Se corrigieron errores de identación.

Cambios de otros archivos

  • Se cambió el tsconfig.json en la parte del target de es6 a es2017.

Pruebas

  • El linter fué pasado exitosamente.
  • El test arrojó una covertura de código alta.
  • Se levanto el proyecto y se probaron las implementaciones a las cuales se tradujó el código para probar su funcionamiento.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11310814827

Details

  • 62 of 65 (95.38%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.01%) to 82.655%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/groups/create.js 62 65 95.38%
Totals Coverage Status
Change from base Build 11261981273: -0.01%
Covered Lines: 22338
Relevant Lines: 25601

💛 - Coveralls

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.

2 participants