Fix config and compile_env stuff for elixir 1.14 warnings#77
Fix config and compile_env stuff for elixir 1.14 warnings#77
Conversation
Pull Request Test Coverage Report for Build c0c9094b0141a2b774d46ce1030dd9f2a2353b22-PR-77
💛 - Coveralls |
|
Probably just remove |
|
Yeah, ideally we should remove all I think there will also be some coveralls ignores to apply if we want to upgrade the latest Elixir version in the matrix 1.12->1.14 because of the extra coverage that we can blame @pmonson711 for 😛 |
|
Unfortunately |
|
@the-mikedavis related to Elixir versions. from Ueberauth https://github.com/ueberauth/.github/blob/master/SECURITY.md
I am not saying you should do it, but 🤷🏻 I would stick to the idea, last 2 or 3 versions |
|
Yeah, updating minimum Elixir version would probably be ok. I like trying to "golf" the needed Elixir version for build since I have run into scenarios in the past where a library forced my hand for updating an application's Elixir version (and that's a bit annoying because then you need to make a larger change just to update the library version than you were otherwise planning). It would probably be ok to at least bump the minimum version we test in CI to 1.9 though so that we can use |
No description provided.