CSCfi / CSCfi/rems

Consider mounting schedulers from `standalone` instead of `handler`.

Open
#2,730 0 comments 0 reactions 0 assignees View on GitHub
Investigate Technical Debt
Dominant language
Clojure
Stars
66
Forks
28
Avg merge
7d 4h
Merged PRs (30d)
2

Description

While reviewing #2728 we noticed that the email (and now application eraser) poller is started in the `rems.handler` namespace with a require. Perhaps it would be nice to start them in the `standalone.clj` and possibly in some of the test fixtures.

The require is rather subtle, because the namespace is only required, but nothing from there is ever referenced. The comment in that line of code "to enable email polling" really seems like a stopgap measure and not a robust way to do this.

See also:
https://github.com/CSCfi/rems/issues/2102

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.