search

workshopper / levelmeup

Level Me Up Scotty! An intro to Node.js databases via a set of self-guided workshops.
Other
271 stars 61 forks source link

Update to latest workshopper #38

Closed talon closed 8 years ago

talon commented 9 years ago

This is a work in progress. I'm currently having difficulty passing the all your base exercise arguments probably because I haven't fully grasped the workshopper(-excercise) module API.

levelmeup run solutions/solution.js
ALL YOUR undefined ARE BELONG TO undefined.

Any guidance is appreciated. :)

talon commented 9 years ago

37

talon commented 9 years ago

The issue I was having seems to be an error brought on by v1.1.0 of workshopper-exercise

martinheidegger commented 9 years ago

Please ping me at gitter. I am looking into it as well would love to chat :smiley:

rvagg commented 9 years ago

soooo ... one of the design decisions I made with the new workshopper was based on the idea that levelmeup should probably not be doing any child-process stdout comparion of submission and solution but rather doing direct inspection of the solution and providing feedback much like MAKE IT MODULAR does in verify.js. It always felt very awkward to push everything through the stdout-comparison model here in levelmeup and I wanted to make it easy to get away from that.

I'm really keen for a rewrite here but unfortunately haven't found any time. If I magically find some I'd love to help out but otherwise, keep slogging away and feel free to ping me if you need input, you may just need to ping a few times because my inbox overfloweth unfortunately.

martinheidegger commented 8 years ago

@talon @rvagg closing this in favor of #79