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

Split update-system into build-system and try-action-system #19

Closed
tangrammer opened this issue Nov 12, 2014 · 3 comments
Closed

Split update-system into build-system and try-action-system #19

tangrammer opened this issue Nov 12, 2014 · 3 comments

Comments

@tangrammer
Copy link

Hi Stuart,
I was wondering if it makes sense to you splitting this update-system behaviour so we could have build-system fn that only assoc-dependencies and try-action fn that try-action

I'm trying to intercept system components and this update-system function is a really security box to try that :) I'd need to have the system build, then intercept selected components and after that try start fn on system

Thanks in advance
Juan

@stuartsierra
Copy link
Collaborator

You can get that by calling update-system with identity as the action.

@tangrammer
Copy link
Author

That's a really (and great) functional solution!
I didn't realise how easy could be to call my own fn wrapping component/start too

Anyway still need a bit more data so I did this PR #20 to be more clear about it

Thanks again!
Juan

@tangrammer
Copy link
Author

Hi Stuart,
I'm closing PR #20 after realising again that this update-system fn with any function and any arguments is a really good (and functional) way to work and transform the system
In my case I get co-dependency relations using a mutable ref of the system (atom) as argument and worked!
https://github.com/tangrammer/co-dependency/blob/master/src/tangrammer/co_dependency.clj#L55

Thanks!
Juan

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

No branches or pull requests

1 participant