Skip to content

Conversation

eksperimental
Copy link
Contributor

No description provided.

@whatyouhide
Copy link
Member

whatyouhide commented May 13, 2017

Would it make sense to have these as env of the :ex_doc application?

lib/ex_doc.ex Outdated
}

@spec default :: map
def default do
Copy link
Member

Choose a reason for hiding this comment

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

We can remove this function as it does not seem to be used anywhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@josevalim done!

@eksperimental
Copy link
Contributor Author

eksperimental commented May 14, 2017

@whatyouhide It would, but I'm afraid we cannot do it, because we need to value in defstuct at compile time, and I think we need the application to be compiled to be able to access the values in mix.exs

I'm getting this error, let me know if you think I'm doing something wrong or it could be done some other way.

** (ArgumentError) application :ex_doc is not loaded, or the configuration parameter :default_config is not set

@josevalim josevalim merged commit deabff5 into elixir-lang:master May 14, 2017
@eksperimental
Copy link
Contributor Author

❤️

@eksperimental eksperimental deleted the default_configuration branch May 14, 2017 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants