Skip to content

Convertir src/groups/search.js de JS a TS#39

Open
freddcastro wants to merge 4 commits intoUSB-CI3715:f24from
freddcastro:translate-groups-search
Open

Convertir src/groups/search.js de JS a TS#39
freddcastro wants to merge 4 commits intoUSB-CI3715:f24from
freddcastro:translate-groups-search

Conversation

@freddcastro
Copy link

En este envío de cambios se resuelve el problema número #28 (corrige #28), que consiste en adaptar el archivo src/posts/data.js a TypeScript.

Para solucionar este problema, se bifurcó el repositorio NodeBB y se procedió a traducir el archivo src/groups/search.js a TypeScript. Luego, se creó un nuevo archivo data.ts en el mismo directorio, donde se reescribió el código original respetando la sintaxis de TypeScript. Finalmente, se generó el archivo JavaScript final (src/groups/search.js) a partir del archivo TypeScript utilizando npx tsc, reemplazando así la versión original.

Se realizaron pruebas locales con npm run lint y npm run test para verificar la calidad y el funcionamiento del código traducido a TypeScript. Además, se inició la aplicación localmente (./nodeBB start) para asegurar que no hubiera errores en tiempo de ejecución. Finalmente, se configuraron pruebas en GitHub Actions para automatizar estas comprobaciones en cada cambio.

Para soportar las funciones async/await, se actualizó el target en tsconfig.json de es6 a es2017. Además, se utilizó el comentario eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call para suprimir temporalmente las advertencias del linter en las asignaciones de variables que involucran archivos aún no migrados a TypeScript.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11309673610

Details

  • 39 of 49 (79.59%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.04%) to 82.625%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/groups/search.js 39 49 79.59%
Totals Coverage Status
Change from base Build 11261981273: -0.04%
Covered Lines: 22323
Relevant Lines: 25583

💛 - 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