https://dagster.io/ logo
Powered by
#announcements
  • f

    freiksenet

    11/09/2018, 3:28 PM
    I'll keep it simple, no yaml schema or anything
  • f

    freiksenet

    11/09/2018, 3:28 PM
    let's see how far this gets me
    👍 1
  • f

    freiksenet

    11/09/2018, 3:28 PM
    I hope I can get smth working on Monday already
  • f

    freiksenet

    11/09/2018, 3:28 PM
    and then spent rest of the week hooking up error checking
  • s

    schrockn

    11/09/2018, 3:43 PM
    👍🏻
  • m

    max

    11/13/2018, 1:56 AM
    TIL circleci has something called an "orb"
  • f

    freiksenet

    11/14/2018, 11:42 AM
    @schrockn I think we should change config format
  • f

    freiksenet

    11/14/2018, 11:42 AM
    instead of
  • f

    freiksenet

    11/14/2018, 11:42 AM
    Copy code
    context:
  contextKey: ....
  • f

    freiksenet

    11/14/2018, 11:43 AM
    Copy code
    context:
   name: contextKey
   otherKeys: ...
  • f

    freiksenet

    11/14/2018, 11:43 AM
    this way we don't need weird hard-coded behaviour
  • f

    freiksenet

    11/14/2018, 11:43 AM
    cause context is always just one object
  • s

    schrockn

    11/14/2018, 11:54 AM
    I’m open to that
  • f

    freiksenet

    11/14/2018, 11:54 AM
    cause it's counterintuitive
  • f

    freiksenet

    11/14/2018, 11:54 AM
    otherwise
  • f

    freiksenet

    11/14/2018, 11:54 AM
    IMO
  • s

    schrockn

    11/14/2018, 11:54 AM
    Just making some coffee and we can discuss in a few
  • f

    freiksenet

    11/14/2018, 11:54 AM
    check out all the improvements I made to editor
  • s

    schrockn

    11/14/2018, 12:03 PM
    checking it out
  • s

    schrockn

    11/14/2018, 12:03 PM
    re: context
  • s

    schrockn

    11/14/2018, 12:04 PM
    that was what it was originally
  • s

    schrockn

    11/14/2018, 12:04 PM
    my concern is that we would have to added tagged unions officially to the type system to handle it
  • s

    schrockn

    11/14/2018, 12:04 PM
    because the type of “otherKeys” will be dependent on the value of the name field
  • f

    freiksenet

    11/14/2018, 12:04 PM
    I think enum for now is enough
  • s

    schrockn

    11/14/2018, 12:04 PM
    which is complicated
  • f

    freiksenet

    11/14/2018, 12:04 PM
    yeah, it is
  • f

    freiksenet

    11/14/2018, 12:04 PM
    hm
  • f

    freiksenet

    11/14/2018, 12:04 PM
    maybe we should have tagged unions then
  • s

    schrockn

    11/14/2018, 12:04 PM
    right
  • s

    schrockn

    11/14/2018, 12:05 PM
    that would be the next option
1...575859...185Latest