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

Iterate over koans and add ns when init repl. #10

Closed
wants to merge 1 commit into from

Conversation

dkinzer
Copy link
Contributor

@dkinzer dkinzer commented Jan 28, 2014

This pull request adds a way to iterate over all the koans in a koans project
much like in functional-koans/clojure-koans#68; except that it is
modularized and handled by the koan-engine.

It is an alternative to functional-koans/clojure-koans#71.

Other benefits are that it would be backwards compatible with any koans
that uses this engine and it does not need to be updated when new koans
are added.

The drawback is that it only solves the issue with vim-fireplace and not eclipse.

This commit adds a way to iterate over all the koans in a koans project
in a modularized way that can be handled by the koan-engine.
@dkinzer
Copy link
Contributor Author

dkinzer commented May 2, 2014

I'm closing this since you decided to go with option #71.

@dkinzer dkinzer closed this May 2, 2014
@dkinzer dkinzer deleted the repl-init branch May 2, 2014 23:44
@dkinzer dkinzer restored the repl-init branch May 3, 2014 00:23
@dkinzer
Copy link
Contributor Author

dkinzer commented May 3, 2014

I just tried the other solution and it doesn't seem to be working for me so I'm keeping this open.

@dkinzer dkinzer reopened this May 3, 2014
@dkinzer
Copy link
Contributor Author

dkinzer commented Jun 16, 2014

This may be related to tpope/vim-fireplace#161.

@trptcolin
Copy link
Member

Pretty sure this was taken care of by the fireplace fix, so closing this - feel free to reopen if I'm mistaken. Thanks!

@trptcolin trptcolin closed this Sep 5, 2014
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

2 participants