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

Cc/test/6/pin elixir version to last known good #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chriscrabtree
Copy link

Hi, first, I appreciate your work on this project! I was starting down a related path, so I was grateful to find your project.

Adding the package to a project revealed some sensitivities to specific versions of Elixir. I confirmed this by running the test suite and noticing issue #6 as well.

This simple PR adds the version info necessary for users of asdf to successfully run the test suite.

My goal with this PR was to establish a working baseline with all tests passing, so that I could then bump to a newer version of Elixir and hopefully help get the project working for those too.

Match the .tool-versions Elixir version to what is already in the
mix.exs file (Elixir 1.10).

Add corresponding Erlang OTP version.

Add a reasonable Java runtime for the PlantUML tasks to pass muster.
This will be especially helpful to those of us who have not kept in
touch with recent developments in the Java world.
Communicate the current sensitivity to Elixir versions so people
better understand errors they may see.
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 this pull request may close these issues.

1 participant