Traducción Exitosa del archivo: src/messaging/edit.js a TypeScript#46
Open
JMLTUnderCode wants to merge 15 commits intoUSB-CI3715:f24from
Open
Traducción Exitosa del archivo: src/messaging/edit.js a TypeScript#46JMLTUnderCode wants to merge 15 commits intoUSB-CI3715:f24from
JMLTUnderCode wants to merge 15 commits intoUSB-CI3715:f24from
Conversation
…. Lint retorna 59 errores de formato que no son corregibles con npm run lint -- --fix.
…nd test exitoso en git.
…or solucionar. Este archivo es funcional en parte dado que la aplicacion logra montarse en servidor y la edicion de mensaje en todos los ambitos es funcional.
…ne para los casos de variables y funciones que provienen de modulos no traducidos a TS. De esta forma ignoramos dichos errores reduciendo a solo 2, uno relacionado al tipo return promise (linea 24) de la funcion getMessagesData y el otro en el tipo de promise retornado(linea 60) debido al mismo problema de la linea 24. El estado actual de este archivo genera un JS funcional para el proyecto NODEBB, sin embargo existe un problema con el uso de los metodos export/import ESModules y CommonJS lo que hace generar un problema de lo mismo en ESLint, sin embargo al corregirlo la app no funcion, por lo que se llevo a usar eslint-disable de import modules en linea 1 del archivo.
…ion es funcional con la App.
…mise y solucion al segundo problema colocando await para el plugins.hooks. Version del archivo: final(funcional). Verificacion de pruebas de Eslint completas en local. Arreglos finales de estilo, comentarios esenciales de informacion personal.
…ror en version local no se manifestó. Version del archivo sigue siendo funcional y pasando todas las pruebas en ESLint local.
…dicho error en version local no se manifestó. Version del archivo sigue siendo funcional y pasando todas las pruebas en ESLint local.
…it.ts), esta version es funcional para la app NodeBB.
…gran a las pruebas y la App, es decir test y lint en local cumplen de forma exitosa. Este commit tiene como funciona, ademas de entregar final, actualizar los archivos tal que el test de github pase la seccion Node (que en local es exitoso).
… a function" al tratar de editar un mensaje. Se cambio el tipo de import a require dado que la version bajo ESModules genera dicho error.
…s with CommonJS genera el error en NodeBB "user.isAdminOrGlobalMod is not a function" al tratar de editar mensajes. La version actual de este archivo pasa ESlint.
Pull Request Test Coverage Report for Build 11318106652Details
💛 - Coveralls |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI3715 - Ingeniería de Software 1 (Proyecto 1B)
Estudiante: Junior Miguel Lara Torres (17-10303)
Proyecto 1B - Traducción a TS del archivo src/messaging/edit.js
Pasos realizados:
import/exportllamadoCommonJSpara evitar problema de no detección de funciones./* eslint-disable @typescript-eslint/no-var-requires */para descartar el error de ESLint al detectar losrequire.// eslint-disable-next-line @typescript-eslint/no-unsafe-assignmentpara descartar el error de ESLint al detectar la asignación deanya losconstmodulares.INTERFACE's:MessagingInterface: Contiene todos los métodos/funciones del sistema de edición de mensajes.MessageData: Contiene los datos de un mensaje.Payload: Contiene la fecha de edición del mensaje y su contenido.edit.js.76:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment85:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call87:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call92:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment108:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment111:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call114:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call120:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment127:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment130:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call136:// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access150:// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment152:// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-calleslint-disable-next-line ...muy largas que sobrepasan los120caracters, se desabilitó dicho error usando// eslint-disable-next-line max-lenen las siguientes lineas:75,91,107,119y126.PD: Esto se puede notar en el apartado anterior, las lineas de código que siguen a cada linea correspondiente mencionada en este inciso son muy largas.
Issue resolves #5