Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Commands::Graph.build and extend Env#command to support it #247

Merged
merged 8 commits into from Jun 1, 2015

Conversation

solnic
Copy link
Member

@solnic solnic commented May 29, 2015

TODO

  • make it possible to use as just like with a normal command
  • move stuff to Graph::ClassInterface for consistency
  • add docs

This allows us to build a command graph without having input yet
…mand graph

...in other words - this thing allows composition of multiple commands
that will process a nested input without the need to provide that input
during composition. So, you can build up a command graph for persisting
object graphs and memoize it and then simply call it with input.

This works with deeply nested input and can be mixed with commands that
return either :one or :many results with just one assumption that the
root command must be configured to return :one result. ie you persist a
user root with tasks, their tags and a user address.
@solnic solnic changed the title [WIP] Add Commands::Graph.build and extend Env#command to support it Add Commands::Graph.build and extend Env#command to support it Jun 1, 2015
solnic added a commit that referenced this pull request Jun 1, 2015
Add Commands::Graph.build and extend `Env#command` to support it
@solnic solnic merged commit b40fb53 into master Jun 1, 2015
@solnic solnic removed the in progress label Jun 1, 2015
@solnic solnic deleted the command-graph-builder branch June 1, 2015 09:04
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling bd4757e on command-graph-builder into ** on master**.

@flash-gordon
Copy link
Member

flash-gordon commented Feb 10, 2017 via email

@solnic
Copy link
Member Author

solnic commented Feb 10, 2017

lol wat :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants