Git Product home page Git Product logo

cdmx-2018-01-fe-markdown's Introduction

Website

This project was generated with Angular CLI version 7.0.3.

Development server

Run ng serve for a dev server. Navigate to http://localhost:4200/. The app will automatically reload if you change any of the source files.

Code scaffolding

Run ng generate component component-name to generate a new component. You can also use ng generate directive|pipe|service|class|guard|interface|enum|module.

Build

Run ng build to build the project. The build artifacts will be stored in the dist/ directory. Use the --prod flag for a production build.

Running unit tests

Run ng test to execute the unit tests via Karma.

Running end-to-end tests

Run ng e2e to execute the end-to-end tests via Protractor.

Further help

To get more help on the Angular CLI use ng help or go check out the Angular CLI README.

cdmx-2018-01-fe-markdown's People

Contributors

danielalab avatar developervilchez avatar diegovelezg avatar fabianbravoa avatar nerdale avatar rosszhena avatar

Watchers

 avatar

cdmx-2018-01-fe-markdown's Issues

Feedback proyecto markdown

  • Tienes un "if" en la línea 8 que utilizas sólo para verificar si es absoluta o no la ruta pero como después utilizas el método "resolve" para convertirla a absoluta creo que podrías quitar ese "if" y dejar sólo el "resolve" para hacer más pequeño tu código.

  • En tu línea 30 declaras con "let" un objeto y en la línea 45 un arreglo, ahí podrías cambiar a "const" porque son constantes debido a que siguen siendo un objeto y un arreglo independientemente de su contenido.

  • Me gusta mucho que hayas usado promesas y sus métodos, intentaste llegar a la solución por medio de ellas y esa era la finalidad del proyecto :)

  • Tu archivo "test-map.js" es un borrado para test, te recomendaría que si sólo es eso no lo subas como parte de tus archivos del proyecto.

Comentarios MDLinks

Hola, algunos comentarios:

  • No encontré tests 😢
  • Esta parte es código muerto
    }).catch(reason => {

    Básicamente porque en tus promesas no estas usando el reject y no va a entrar a ese caso.
  • No es necesario que hagas esta asígnación
    resolve(fileName = path.resolve(route));

    De todas maneras en el otro lado lo va a recibir como filename, a menos que sea parte de un objeto. Técnicamente todo esta funcionando porque esa asignación hace de filename una variable global, pero no es la idea.
  • Aquí
    let mypromesas = [];

    Así le hagas push al arreglo, su referencia es la misma y no se modifica, por lo que en realidad debería seguir siendo un const y no un let.

Feedback markdown from Diana

Me llamo mucho la atención que decidiste convertir convertir el archivo md a html, en lo personal a mi se me hizo más complicada esa forma, y reconozco tu esfuerzo por buscar información, entenderla y aplicarla.
También me gusto que usaste Promise.all eso habla de tu capacidad de aprender y buscar más información de la proporcionada. Aunque creo que puedes seguir implementando las promesas en tu código.
Te dejo unos links que espero te sirvan para que trabajes con promesas
promesas-google
promesas-medium

Creo que la línea 8 se podría eliminar ya que en tu línea 9 con path.resolve ya estas cambiando la ruta a absoluta y ya no es necesario primero corroborar si es absoluta para después convertirla. Te ahorrarías un paso :) o bien si quieres dejarla creo que la sintaxis podría quedar así if (!path.isAbsolute(route)) ya que ahí le dices si el archivo es diferente a absoluto (relativo) lo convierta a absoluto con path.resolve.
Espero darme a entender y no confundirte, sino con confianza puedes acercarte a preguntarme.

También note que implementaste test, excelente! Sólo quitaría todo el código muerto (comentado) de ese archivo.
Te dejo unos links que creo puedes implementar en tus tests después de que implementes las promesas
test-promesas

Feedback

Me gusta que usaste promesas y que la mayoría de tus funciones son puras. Podrías simplificar cada una para que sean funciones que sólo hagan una cosa por cada función. Nunca he usado promise.all pero me llama la atención que controlaste el flujo de tu funcionalidad con esta promesa.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.