Christopher De Cairos cadecairos

Organizations

@mozilla
cadecairos commented on pull request hapijs/bassmaster#45
@cadecairos

I think we're going to want to use Hoek.reach VS JSONPath see #56

@cadecairos
  • @cadecairos b2b2bc0
    Note usage of Hoek.reach in README
cadecairos opened pull request hapijs/bassmaster#56
@cadecairos
Use Hoek.reach for retrieving values from objects
1 commit with 2 additions and 8 deletions
cadecairos commented on issue hapijs/bassmaster#50
@cadecairos

This can be better accomplished with Hoek.reach

@cadecairos
Allow using nested object values when piplining
cadecairos commented on issue hapijs/bassmaster#44
@cadecairos

It looks like we may just want to go with Hoek.reach instead of jsonpath, which would allow us to do this in a 1.9 without breaking anything.

cadecairos commented on pull request hapijs/bassmaster#51
@cadecairos

I wasn't aware of it, but it sounds better than having to maintain a new function

cadecairos commented on issue hapijs/bassmaster#44
@cadecairos

I think it'll be better to make it a 2.0 feature, since I don't think it's ideal to have two different syntaxes for two similar functions.

cadecairos commented on pull request mozilla/publish.webmaker.org#102
@cadecairos

R+

@cadecairos
Fix promise failure paths for create and remix
2 commits with 101 additions and 14 deletions
@cadecairos
  • @cadecairos dc19e9a
    Fix up remix promises, add tests for coverage
@cadecairos
  • @cadecairos e2b6307
    clean up create project promises
@cadecairos

Caching being handled in #37

cadecairos commented on pull request mozilla/api.webmaker.org#107
@cadecairos

So, I wonder if I should add tests, considering bassmaster has tests for all its functionality, and basically just injects into existing and tested…

@cadecairos
@cadecairos
@cadecairos
  • @cadecairos a8f5c30
    Update version in package.json to 1.8.0
@cadecairos
cadecairos merged pull request hapijs/bassmaster#55
@cadecairos
Added documentation about auth config
1 commit with 1 addition and 0 deletions
@cadecairos
Document the auth option
cadecairos commented on pull request hapijs/bassmaster#55
@cadecairos

@Torsph Awesome, thanks :tada: :icecream: :hamburger:

@cadecairos

So, I've become the maintainer of hapijs/bassmaster There's a few things that need attention, then I'll tag and ship 1.8.0

cadecairos commented on issue hapijs/bassmaster#32
@cadecairos

@arb can you clarify what needs to be updated here? Is it all the server setup that should be moved outside the describe block? (https://github.co…

cadecairos commented on issue hapijs/bassmaster#29
@cadecairos

@jardakotesovec Can you provide a use case for manually parsing the response data after inject VS using the pre-parsed result? I don't fully unders…

@cadecairos
cadecairos merged pull request hapijs/bassmaster#46
@cadecairos
Update caolan/async to 1.3.x from 0.9.x
1 commit with 1 addition and 1 deletion
cadecairos commented on pull request hapijs/bassmaster#46
@cadecairos

Yeah, that sounds okay to me.

@cadecairos
Can you pass in options.auth to config.auth in the setup?