Jsdoc support - Githubissues

jauco / jester

Get your project tested and out there with minimal fuss.

https://trello.com/b/GLFKT2X4/jester

BSD 3-Clause "New" or "Revised" License

3 stars 3 forks source link

Jsdoc support #5

Closed Lutger closed 10 years ago

Lutger commented 10 years ago

[x] readme updaten met jsdoc instructies
[x] readme updaten met cross-platform opmerking
[x] error: jsdoc failed! { [Error: Command failed: FATAL: Unable to load template: Cannot find module 'd:\Git\myApp\node_modules\jester-tester\node_modules\jsdoc\templates/publish'
[x] runAllTests: falende lints moeten zorgen voor een nonzero exitcode
[x] jester watch: falende lints moeten het proces afbreken
[x] npm shrinkmap
[x] jester-watch: rebuild+tests werken niet meer parallel
[x] versie bump

jauco commented 10 years ago

Misschien goed om ook http://eslint.org/docs/rules/valid-jsdoc.html aan te zetten

barbarah commented 10 years ago

In de readme is niks over jsdoc terug te vinden. Wat ik op zijn mist had verwacht:

onder het kopje 'Creating a project' staat een lijstje met door jester-init gegenereerde mappen. ./doc/api ontbreekt.
Een korte uitleg waar jsdoc.conf voor is
Wanneer wordt de api gegenereerd? Bij jester-batch, watch en doc?
Een link naar jsdoc documentatie
Eventueel een voorbeeld hoe te gebruiken

barbarah commented 10 years ago

Ik krijg een error:

jsdoc failed! { [Error: Command failed: FATAL: Unable to load template: Cannot find module 'd:\Git\myApp\node_modules\jester-tester\node_modules\jsdoc\templates/publish'

barbarah commented 10 years ago

Kudos voor de heldere code. Kan het zonder veel kennis van node en alle dependencies goed lezen. Ik begrijp nu goed hoe de nieuwe jester werkt. Ik heb niet volledig ingelezen in alle dependencies, dus ik kan niet goed beoordelen of deze op de juiste manier gebruik zijn. @jauco kan jij er ook nog een keer naar kijken?

Wat mij betreft kan het na het oplossen van de twee bovenstaande opmerkingen ge-merge worden.

jauco commented 10 years ago

Ik had hem al snel doorgekeken afgelopen vrijdag, maar nog niet nauwkeurig naar bugs gezocht en getest. Alvast twee opmerkingen toegevoegd zodat er alvast naar gekeken kan worden. Ik probeer ergens deze week de code goed te reviewen.

Lutger commented 10 years ago

@barbarah: ik heb de jsdoc error gefixed, je kan na een update van jester het bestand jsdoc.conf weggooien en dan jester-init nog een keer draaien. Ik heb ook de readme geupdate, hoor graag of het nog beter zou kunnen, vind het lastig om dit helder te verwoorden.

barbarah commented 10 years ago

Ik heb de readme helemaal doorgenomen. Helder en ik kreeg geen errors. Van mij mag deze pull request ge-merged worden!

Lutger commented 10 years ago

@jauco, versie is al van 1.0.0 naar 1.1.0 opgehoogd in deze pull request. Kan je checken of het parallel builden + testen correct is gedaan? Dan is ie klaar voor beta testen :)