Skip to content

Fix arity of :log-fn passed to clj-reload.core/reload#28

Merged
weavejester merged 1 commit intoweavejester:masterfrom
rihteri:log_fn_arity
Mar 23, 2026
Merged

Fix arity of :log-fn passed to clj-reload.core/reload#28
weavejester merged 1 commit intoweavejester:masterfrom
rihteri:log_fn_arity

Conversation

@rihteri
Copy link
Copy Markdown
Contributor

@rihteri rihteri commented Jan 2, 2026

Apparently, reload/reload would like function that works with any arity, as documented here: https://github.com/tonsky/clj-reload/blob/61c6fa77855efc0b4d052f06c7cc8b2b8817c4aa/src/clj_reload/core.clj#L338

With the single-parameter arity I'm getting an exception from logging that masks my actual problem:

Execution error (ArityException) at clj-reload.util/log (util.clj:15).
Wrong number of args (3) passed to: integrant.repl/reload/fn--31852

@weavejester
Copy link
Copy Markdown
Owner

Thanks for the PR and apologies for the delay -- for some reason I thought I'd already merged this! Can you change the commit message to:

Fix arity of :log-fn

(In order to adhere to the contributing guidelines.)

@rihteri
Copy link
Copy Markdown
Contributor Author

rihteri commented Mar 23, 2026

Changed

@rihteri rihteri changed the title fix arity of :log-fn passed to clj-reload.core/reload Fix arity of :log-fn passed to clj-reload.core/reload Mar 23, 2026
@weavejester weavejester merged commit bf2943a into weavejester:master Mar 23, 2026
1 check passed
@weavejester
Copy link
Copy Markdown
Owner

Much obliged!

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.

2 participants