by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Nicolas Ritouet
    @NicolasRitouet
    feature request, either send a pull request or start a discussion somewhere else (google groups?), and requests for help: forward to stackoverflow
    if you have time, can you read it and let me know what you think?
    Nicolas Ritouet
    @NicolasRitouet
    if it’s not a bug, let’s redirect support question to stackoverflow: http://stackoverflow.com/questions/tagged/deployd
    Eric Fong
    @ericfong
    Just rebased few PR. the important PR #496 launched to master.
    And I just find that deployd is good to run under iojs-1.5.0. So Great!
    And if you have time, please take a look at #523 . A bit not sure what will happen to other middleware or stream funcs
    Andrei Alecu
    @andreialecu
    I find myself needing to validate the schema of inner objects in my records
    I think it would be a nice idea for the roadmap to use json-schema for describing collections in the future
    it lets you do a lot better validation
    regex patterns, one of, min, max
    Andrei Alecu
    @andreialecu
    Nicolas Ritouet
    @NicolasRitouet
    @andreialecu looks good
    Eric Fong
    @ericfong
    Any thoughts on deployd/deployd#315 ?
    Nicolas Ritouet
    @NicolasRitouet
    Hi guys
    sorry, I was sick the last few days
    give me a few days and we’ll take some time to cleanup all these PR
    Eric Fong
    @ericfong
    Take care
    Andrei Alecu
    @andreialecu

    Git Commit Guidelines

    We have very precise rules over how our git commit messages can be formatted. This leads to more readable messages that are easy to follow when looking through the project history. But also, we use the git commit messages to generate the AngularJS change log.

    :)
    I'm thinking about making my first merge, with this: deployd/deployd#535
    unless someone else wants to
    Nicolas Ritouet
    @NicolasRitouet
    go ahead @andreialecu
    what do you guys think about this one: deployd/deployd#490
    can we find a better solution?
    Nicolas Ritouet
    @NicolasRitouet
    @andreialecu you are on fire ! :clap:
    Eric Fong
    @ericfong
    I would like to have a temp solution
    Andrei Alecu
    @andreialecu
    I think we can merge it like that
    but has someone tested it manually?
    ensured it works with either the function call and/or the dpd.js script element?
    Eric Fong
    @ericfong
    Personally, I have done some, but may need to do one for rebasing to the new master
    Andrei Alecu
    @andreialecu
    it may be tested via sinon
    with some mocks maybe
    mock io.connect and check if it's called with proper parameters
    and the XHR object
    I'll take a stab at it
    and send you a PR with tests
    if I can figure something out
    send it to your branch I mean ericfong:urlBase
    Eric Fong
    @ericfong
    Yes. please. sleep first
    Andrei Alecu
    @andreialecu
    @ericfong here you go: ericfong/deployd#1
    Andrei Alecu
    @andreialecu
    hey guys, so I just gutted the CLI out of deployd and made this: https://github.com/andreialecu/deployd-cli
    Andrei Alecu
    @andreialecu
    I think this is the only thing that should be installed globally, with deployd installed locally
    grunt does the same
    the cli is global, but grunt is local in node_modules
    I changed dpd create in this to create a package.json that has deployd in it, and then run npm install so it seems to all just work
    Andrei Alecu
    @andreialecu
    also, I made a fork without the cli here: https://github.com/andreialecu/deployd/tree/remove-cli
    Eric Fong
    @ericfong
    WOW, Good move for separating dpd cli and deployd core
    Andrei Alecu
    @andreialecu
    I'll work on it some more and maybe we can move the deployd-cli repo to deployd/
    and merge the removal