search

arch-js / arch-cli

Command line tool for the Arch framework
1 stars 2 forks source link

Update CLI to the latest Reflex API #5

Closed charypar closed 9 years ago

charypar commented 9 years ago

redbadger/reflex#7 introduces extra arguments to reflex's server.run, we need to update the CLI to make the right call.

tabazevedo commented 9 years ago

Will update server.js in generator-reflex

charypar commented 9 years ago

Are we still going through server.js? It was meant as a temporary thing, I assumend the CLI would just make the same set of calls and we'll get rid of it eventually.

tabazevedo commented 9 years ago

Hmm going through server.js is a workaround to the hotloading of server code. Eventually yeah it would be great to skip it entirely, but IMO getting the other functionality of the CLI working should take priority

charypar commented 9 years ago

True. So... could we at least move server.js into the CLI and pass the necessary paths into it through ENV? I'm just trying to remove as much cruft from the app itself as possible. The less convention we introduce, the better.

tabazevedo commented 9 years ago

Yeah I was thinking that something like that would be more in tune with what we had in mind for Reflex. I'll figure it out today i think, sounds more interesting than using Yeoman programmatically

tabazevedo commented 9 years ago

@charypar closing this, fixed in https://github.com/redbadger/reflex-cli/pull/6