Skip to content

Traducción del archivo NodeBB/scr/messaging/unread.js de JS a TS#35

Open
A6575 wants to merge 7 commits intoUSB-CI3715:f24from
A6575:traduccion-a-ts-scr/messaging/unread.js
Open

Traducción del archivo NodeBB/scr/messaging/unread.js de JS a TS#35
A6575 wants to merge 7 commits intoUSB-CI3715:f24from
A6575:traduccion-a-ts-scr/messaging/unread.js

Conversation

@A6575
Copy link

@A6575 A6575 commented Oct 11, 2024

Proyecto 1B: Traducción del archivo src/messaging/unread.js de JS a TS

(resolves #6) En el archivo se realizaron los siguientes cambios para hacer una traducción completa de JS a TS:

  • Modificación de los import para que sean acorde a la sintaxis de TypeScript. Se le agregó el comentario: /* eslint-disable import/no-import-module-exports */ para evitar errores de importación de archivos no traducidos a TS.
  • Se agregó las interfaces Messaging, TimestampData y MidData para el tipado de las funciones y asignaciones de variables.
  • A lo largo del archivo, se tuvo que agregar los comentarios eslint-disable con los errores de @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access y @typescript-eslint/no-unsafe-return, dado que los mismos están involucrados a funciones de otros archivos que no están traducidas a TS.
  • Se realizaron modificaciones para solventar los errores que ESlint reportaba. Entre estos, se buscó eliminar el uso de any, mejorar el tipado de las variables y/o funciones, errores de identación, errores de espaciado y errores de longitud de línea.

Important

Actualmente, con el archivo unread.js generado al compilar la traducción unread.ts la página es 100% funcional

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11300737629

Details

  • 48 of 59 (81.36%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.006%) to 82.66%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/messaging/unread.js 48 59 81.36%
Totals Coverage Status
Change from base Build 11261981273: -0.006%
Covered Lines: 22342
Relevant Lines: 25602

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

Convirtiendo src/messaging/unread.js de JS a TS

2 participants