-
Notifications
You must be signed in to change notification settings - Fork 36
ci(github): ci housekeeping #76
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
base: master
Are you sure you want to change the base?
Conversation
List of changes: - add latest Elixir 1.17 / Erlang OTP 27 to CI matrix - bump all deps and github/actions - code format based on elixir 1.17 - replace deprecated `:plt_add_deps` option - run CI on push to master or pull request only - set min Elixir requirement to 1.13 - use Ubuntu-20.04 as base OS image due to deprecation of Ubuntu 18.04
| push: | ||
| branches: | ||
| - "*" | ||
| - "master" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove duplicate CI jobs.
| description: "A GraphQL client for elixir.", | ||
| version: @version, | ||
| elixir: "~> 1.7", | ||
| elixir: "~> 1.13", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest Credo requires 1.13.
| elixir: "~> 1.13", | ||
| build_embedded: Mix.env() == :prod, | ||
| start_permanent: Mix.env() == :prod, | ||
| test_coverage: [tool: Coverex.Task], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unsure whether you want to retain Coverex, https://github.com/alfert/coverex, which have not been updated for 6 years.
|
@uesteibar Any updates on this? |
List of changes:
:plt_add_depsoption