Skip to content

Commit

Permalink
Improve CI and fix warnings (#53)
Browse files Browse the repository at this point in the history
  • Loading branch information
tiagoefmoraes authored Jan 12, 2024
1 parent 371c576 commit 5306d4f
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 9 deletions.
29 changes: 22 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
push:
branches:
- main
- misc

jobs:
mix_test:
Expand All @@ -14,28 +15,42 @@ jobs:
fail-fast: false
matrix:
include:
- elixir: 1.13.4
otp: 24.0
- elixir: 1.13.4
otp: 25.0
- elixir: 1.14.0
otp: 25.0
- elixir: '1.13.4'
otp: '24.0'
- elixir: '1.13.4'
otp: '25.0'
- elixir: '1.14.0'
otp: '25.0'
check_formatted: true
warnings_as_errors: true
- elixir: '1.15.2'
otp: '26.0'
check_formatted: true
warnings_as_errors: true
env:
MIX_ENV: test
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- uses: erlef/setup-beam@v1
with:
otp-version: ${{matrix.otp}}
elixir-version: ${{matrix.elixir}}
- name: Deps and _build cache
uses: actions/cache@v3
id: deps-cache
with:
path: |
deps
_build
key: ${{ runner.os }}-${{ runner.arch }}-${{ matrix.otp }}-${{ matrix.elixir }}-${{ hashFiles('mix.lock') }}
- name: Install Dependencies
if: steps.deps-cache.outputs.cache-hit != 'true'
run: |
mix local.hex --force
mix local.rebar --force
mix deps.get --only test
- run: mix deps.compile
if: steps.deps-cache.outputs.cache-hit != 'true'
- run: mix format --check-formatted
if: matrix.check_formatted
- run: mix compile
Expand Down
4 changes: 2 additions & 2 deletions config/config.exs
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import Mix.Config
import Config

config :phoenix, :json_library, Jason
config :phoenix, :stacktrace_depth, 20

config :logger, level: :warn
config :logger, level: :warning
config :logger, :console, format: "[$level] $message\n"

# Configure esbuild (the version is required)
Expand Down

0 comments on commit 5306d4f

Please sign in to comment.