-
Notifications
You must be signed in to change notification settings - Fork 13
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
before_closing_body_tag option #44
Comments
Agreed. We need to support this and with as little friction as possible. So, like you could have a module that's just for doing this kind of stuff, and have it excluded from your release, but that feels like friction. I'm asking about if supporting anon functions in rebar.config is even possible. |
It is not possible in rebar.config because it uses |
I don't think anon functions in rebar.config are going to work, but I do have an idea that would be frictionless. What about an option for in ex_doc config (within rebar.config) that looks something like :
Then rebar3_ex_doc can make an anon function using this data. I'm not sure exactly how we would dump this to config for ex doc, as I've only ever used mix.exs to accomplish this, so have to make sure dumping it to config at that point is doable. Edit: ex_doc uses file:consult/1 when reading in a config file that is not exs, so that would be a no go. But worth pointing out a temporary work around is indeed the new support for specifying a |
So, option 1 is to allow Also, option 3 is to just use some module which is excluded from compilation... Rebar3 runs ex_doc in |
@MarkoMin Two PRs were done to ex_doc to facilitate this better. Waiting for a new version of ex_doc to be cut, then a new version of this plugin will be released. |
Hey guys, this fell on the floor. v0.2.18 has been released with ex_doc v0.29.4. Let me know if you have issues ❤️ |
Hi,
I just tried rebar3_ex_doc and I'd like to render mermaid graphs. I got it working by placing {before_closing_body_tag, fun my_mod:html/1} option. However, there are 2 problems with this approach:
p.s. I'm open to contribution
The text was updated successfully, but these errors were encountered: