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

Errors are swallowed during recompilation #14

Closed
CrowdHailer opened this issue Oct 29, 2017 · 3 comments · Fixed by #33
Closed

Errors are swallowed during recompilation #14

CrowdHailer opened this issue Oct 29, 2017 · 3 comments · Fixed by #33

Comments

@CrowdHailer
Copy link
Collaborator

I have a routing DSL that deliberately throws errors during the compilation step if controllers are not defined correctly. The error messages are very useful and are shown if manually calling recompile.

e.g.

iex(1)> recompile
Compiling 1 file (.ex)

== Compilation error in file lib/www.ex ==
** (RuntimeError) module WWW.HomePagezz is not loaded
    (raxx) lib/raxx/router.ex:51: anonymous fn/3 in Raxx.Router."MACRO-__using__"/2
    (elixir) lib/enum.ex:1826: Enum."-reduce/3-lists^foldl/2-0-"/3
    (raxx) expanding macro: Raxx.Router.__using__/1
    lib/www.ex:9: WWW (module)
    (elixir) expanding macro: Kernel.use/2
    lib/www.ex:9: WWW (module)
    (tokumei) expanding macro: Raxx.Blueprint.__using__/1
    lib/www.ex:9: WWW (module)

When using ExSync. There is simply no output if recompilation fails. Is there a configuration that allows me to show the same errors as I would get for calling recompile.

@falood
Copy link
Owner

falood commented Oct 30, 2017

I will check why it's missed 🙂 thanks

@lATAl
Copy link

lATAl commented Oct 21, 2018

Hi @falood, any updates for �this issue?

axelson added a commit to axelson/exsync that referenced this issue May 15, 2020
Also refactor logging to support debug and warn

Fixes falood#14
axelson added a commit to axelson/exsync that referenced this issue May 23, 2020
Also refactor logging to support debug and warn

Fixes falood#14
@axelson
Copy link
Collaborator

axelson commented May 23, 2020

@CrowdHailer this is improved with #33. Please try it out on the master branch.

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 a pull request may close this issue.

4 participants