Skip to content

Add clojure templates#68

Open
ellnix wants to merge 10 commits into
Andriamanitra:mainfrom
ellnix:add-clojure-templates
Open

Add clojure templates#68
ellnix wants to merge 10 commits into
Andriamanitra:mainfrom
ellnix:add-clojure-templates

Conversation

@ellnix

@ellnix ellnix commented Apr 28, 2024

Copy link
Copy Markdown
Collaborator

This is meant to go along with #65

Split off for easier review.

@ellnix ellnix force-pushed the add-clojure-templates branch from 5bfb6bc to b5a6d3e Compare April 28, 2024 15:48
@ellnix ellnix force-pushed the add-clojure-templates branch from b5a6d3e to cdace3e Compare April 28, 2024 15:50
@ellnix ellnix marked this pull request as ready for review April 28, 2024 15:50

@Andriamanitra Andriamanitra left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to keep the ns and defn main for compatibility with CodinGame? Writing code on the top level would "just work" with bb sol.clj or clj -M sol.clj, but running the main function requires a more complicated incantation. Solutions don't ever need to use their args so having a main function doesn't really add any other value than CodinGame compatibility.

Comment thread config/stub_templates/clojure/stub_config.toml
Comment thread config/stub_templates/clojure/write.clj.jinja Outdated
Comment thread config/stub_templates/clojure/write_join.clj.jinja Outdated
Comment thread config/stub_templates/clojure/stub_config.toml Outdated
@ellnix

ellnix commented May 7, 2024

Copy link
Copy Markdown
Collaborator Author

I summon the clojure and template master: @daxida

Comment thread config/stub_templates/clojure/stub_config.toml Outdated
@ellnix ellnix force-pushed the add-clojure-templates branch from 3897e80 to 3309a0a Compare May 8, 2024 17:17
@ellnix ellnix requested a review from Andriamanitra May 8, 2024 17:18
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.

3 participants