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

Project structure for newcomers #121

Closed
tejasbubane opened this issue Jul 23, 2016 · 4 comments
Closed

Project structure for newcomers #121

tejasbubane opened this issue Jul 23, 2016 · 4 comments

Comments

@tejasbubane
Copy link
Member

tejasbubane commented Jul 23, 2016

I am pretty new the clojure and just submitted the hello-world. The running the tests section on language page does not mention where to put my solution file, so initially I assumed it would be the problem directory itself. This gave me the error:

Could not locate hello_world__init.class or hello_world.clj on classpath

So I finally put it inside test/ assuming it is the tests that require the file and tests ran successfully. Later found out that exercism-clojure uses to the "real-world project structure" here #84.

I think the directory structure should be mentioned in running the tests section. Also if possible, creating an empty file inside src directory would help, so that people know the source code goes there.

@yurrriq
Copy link
Member

yurrriq commented Jul 23, 2016

Hey, thanks for the feedback! I can see how that could be confusing to newcomers.

Would you be willing to submit a PR with "stub" solution files? I think that's a great idea.

Update: WIP #122

tejasbubane added a commit to tejasbubane/xclojure that referenced this issue Sep 19, 2016
Hello world was already added in exercism#122, add to next 4 exercises
(to make it 5)

Fixes exercism#121
@kevgathuku
Copy link
Contributor

kevgathuku commented Oct 15, 2016

Hey @tejasbubane @yurrriq Thanks for your work on this.
Would it make sense to add the stub files for all exercises, or to make them a requirement when adding a new exercise?
It would make the whole thing more consistent, rather than some exercises having it and others not having it.
I could add them if needed.

@canweriotnow
Copy link
Contributor

@kevgathuku @tejasbubane @yurrriq I think making stub solution files mandatory for all tracks - esp. using "real-world" leiningen projects - is a must-have. On the xscheme and xelisp tracks we always provide stub files to make sure the exercist has the proper requires/includes and file format (esp. for elisp so flycheck doesn't have a conniption)... having a stub file with the ns macro declaration and any necessary requires makes complete sense. I'd be happy to create and self-assign an issue to take this on if desired.

@yurrriq
Copy link
Member

yurrriq commented Oct 16, 2016

Sounds good to me then.

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

4 participants