Skip to content

Commit 14494e6

Browse files
cabolcabol
committed
[#7] Migrate adapter to Nebulex v3
1 parent a659dba commit 14494e6

34 files changed

+1302
-651
lines changed

‎.credo.exs‎

Lines changed: 176 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,188 @@
1+
# This file contains the configuration for Credo and you are probably reading
2+
# this after creating it with `mix credo.gen.config`.
3+
#
4+
# If you find anything wrong or unclear in this file, please report an
5+
# issue on GitHub: https://github.com/rrrene/credo/issues
6+
#
17
%{
8+
#
9+
# You can have as many configs as you like in the `configs:` field.
210
configs: [
311
%{
12+
#
13+
# Run any config using `mix credo -C <name>`. If no config name is given
14+
# "default" is used.
15+
#
416
name: "default",
17+
#
18+
# These are the files included in the analysis:
519
files: %{
6-
included: ["lib/", "src/", "test/", "benchmarks/"],
7-
excluded: [~r"/_build/", ~r"/deps/"]
20+
#
21+
# You can give explicit globs or simply directories.
22+
# In the latter case `**/*.{ex,exs}` will be used.
23+
#
24+
included: [
25+
"lib/",
26+
"src/",
27+
"test/",
28+
"benchmarks/",
29+
"web/",
30+
"apps/*/lib/",
31+
"apps/*/src/",
32+
"apps/*/test/",
33+
"apps/*/web/"
34+
],
35+
excluded: [~r"/_build/", ~r"/deps/", ~r"/node_modules/"]
836
},
37+
#
38+
# Load and configure plugins here:
39+
#
40+
plugins: [],
41+
#
42+
# If you create your own checks, you must specify the source files for
43+
# them here, so they can be loaded by Credo before running the analysis.
44+
#
45+
requires: [],
46+
#
47+
# If you want to enforce a style guide and need a more traditional linting
48+
# experience, you can change `strict` to `true` below:
49+
#
50+
strict: true,
51+
#
52+
# To modify the timeout for parsing files, change this value:
53+
#
54+
parse_timeout: 5000,
55+
#
56+
# If you want to use uncolored output by default, you can change `color`
57+
# to `false` below:
58+
#
959
color: true,
60+
#
61+
# You can customize the parameters of any check by adding a second element
62+
# to the tuple.
63+
#
64+
# To disable a check put `false` as second element:
65+
#
66+
# {Credo.Check.Design.DuplicatedCode, false}
67+
#
1068
checks: [
11-
# Readability Checks
12-
{Credo.Check.Readability.MaxLineLength, priority: :low, max_length: 100},
69+
#
70+
## Consistency Checks
71+
#
72+
{Credo.Check.Consistency.ExceptionNames, []},
73+
{Credo.Check.Consistency.LineEndings, []},
74+
{Credo.Check.Consistency.ParameterPatternMatching, []},
75+
{Credo.Check.Consistency.SpaceAroundOperators, []},
76+
{Credo.Check.Consistency.SpaceInParentheses, []},
77+
{Credo.Check.Consistency.TabsOrSpaces, []},
1378

14-
# TODO and FIXME do not cause the build to fail
15-
{Credo.Check.Design.TagTODO, exit_status: 0},
16-
{Credo.Check.Design.TagFIXME, exit_status: 0}
79+
#
80+
## Design Checks
81+
#
82+
# You can customize the priority of any check
83+
# Priority values are: `low, normal, high, higher`
84+
#
85+
{Credo.Check.Design.AliasUsage, [priority: :low]},
86+
# You can also customize the exit_status of each check.
87+
# If you don't want TODO comments to cause `mix credo` to fail, just
88+
# set this value to 0 (zero).
89+
#
90+
{Credo.Check.Design.TagTODO, [exit_status: 0]},
91+
{Credo.Check.Design.TagFIXME, [exit_status: 0]},
92+
93+
#
94+
## Readability Checks
95+
#
96+
{Credo.Check.Readability.AliasOrder, []},
97+
{Credo.Check.Readability.FunctionNames, []},
98+
{Credo.Check.Readability.LargeNumbers, []},
99+
{Credo.Check.Readability.MaxLineLength, [priority: :low, max_length: 100]},
100+
{Credo.Check.Readability.ModuleAttributeNames, []},
101+
{Credo.Check.Readability.ModuleDoc, []},
102+
{Credo.Check.Readability.ModuleNames, []},
103+
{Credo.Check.Readability.ParenthesesInCondition, []},
104+
{Credo.Check.Readability.ParenthesesOnZeroArityDefs, []},
105+
{Credo.Check.Readability.PredicateFunctionNames, []},
106+
{Credo.Check.Readability.PreferImplicitTry, []},
107+
{Credo.Check.Readability.RedundantBlankLines, []},
108+
{Credo.Check.Readability.Semicolons, []},
109+
{Credo.Check.Readability.SpaceAfterCommas, []},
110+
{Credo.Check.Readability.StringSigils, []},
111+
{Credo.Check.Readability.TrailingBlankLine, []},
112+
{Credo.Check.Readability.TrailingWhiteSpace, []},
113+
{Credo.Check.Readability.UnnecessaryAliasExpansion, []},
114+
{Credo.Check.Readability.VariableNames, []},
115+
116+
#
117+
## Refactoring Opportunities
118+
#
119+
{Credo.Check.Refactor.CondStatements, []},
120+
{Credo.Check.Refactor.CyclomaticComplexity, [max_complexity: 40]},
121+
{Credo.Check.Refactor.FunctionArity, []},
122+
{Credo.Check.Refactor.LongQuoteBlocks, [max_line_count: 200]},
123+
# {Credo.Check.Refactor.MapInto, []},
124+
{Credo.Check.Refactor.MatchInCondition, []},
125+
{Credo.Check.Refactor.NegatedConditionsInUnless, []},
126+
{Credo.Check.Refactor.NegatedConditionsWithElse, []},
127+
{Credo.Check.Refactor.Nesting, []},
128+
{Credo.Check.Refactor.UnlessWithElse, []},
129+
{Credo.Check.Refactor.WithClauses, []},
130+
131+
#
132+
## Warnings
133+
#
134+
{Credo.Check.Warning.ApplicationConfigInModuleAttribute, []},
135+
{Credo.Check.Warning.BoolOperationOnSameValues, []},
136+
{Credo.Check.Warning.ExpensiveEmptyEnumCheck, []},
137+
{Credo.Check.Warning.IExPry, []},
138+
{Credo.Check.Warning.IoInspect, []},
139+
# {Credo.Check.Warning.LazyLogging, []},
140+
{Credo.Check.Warning.MixEnv, false},
141+
{Credo.Check.Warning.OperationOnSameValues, []},
142+
{Credo.Check.Warning.OperationWithConstantResult, []},
143+
{Credo.Check.Warning.RaiseInsideRescue, []},
144+
{Credo.Check.Warning.UnusedEnumOperation, []},
145+
{Credo.Check.Warning.UnusedFileOperation, []},
146+
{Credo.Check.Warning.UnusedKeywordOperation, []},
147+
{Credo.Check.Warning.UnusedListOperation, []},
148+
{Credo.Check.Warning.UnusedPathOperation, []},
149+
{Credo.Check.Warning.UnusedRegexOperation, []},
150+
{Credo.Check.Warning.UnusedStringOperation, []},
151+
{Credo.Check.Warning.UnusedTupleOperation, []},
152+
{Credo.Check.Warning.UnsafeExec, []},
153+
154+
#
155+
# Checks scheduled for next check update (opt-in for now, just replace `false` with `[]`)
156+
157+
#
158+
# Controversial and experimental checks (opt-in, just replace `false` with `[]`)
159+
#
160+
{Credo.Check.Consistency.MultiAliasImportRequireUse, false},
161+
{Credo.Check.Consistency.UnusedVariableNames, false},
162+
{Credo.Check.Design.DuplicatedCode, false},
163+
{Credo.Check.Readability.AliasAs, false},
164+
{Credo.Check.Readability.BlockPipe, false},
165+
{Credo.Check.Readability.ImplTrue, false},
166+
{Credo.Check.Readability.MultiAlias, false},
167+
{Credo.Check.Readability.SeparateAliasRequire, false},
168+
{Credo.Check.Readability.SinglePipe, false},
169+
{Credo.Check.Readability.Specs, false},
170+
{Credo.Check.Readability.StrictModuleLayout, false},
171+
{Credo.Check.Readability.WithCustomTaggedTuple, false},
172+
{Credo.Check.Refactor.ABCSize, false},
173+
{Credo.Check.Refactor.AppendSingleItem, false},
174+
{Credo.Check.Refactor.DoubleBooleanNegation, false},
175+
{Credo.Check.Refactor.ModuleDependencies, false},
176+
{Credo.Check.Refactor.NegatedIsNil, false},
177+
{Credo.Check.Refactor.PipeChainStart, false},
178+
{Credo.Check.Refactor.VariableRebinding, false},
179+
{Credo.Check.Warning.LeakyEnvironment, false},
180+
{Credo.Check.Warning.MapGetUnsafePass, false},
181+
{Credo.Check.Warning.UnsafeToAtom, false}
182+
183+
#
184+
# Custom checks can be created using `mix credo.gen.check`.
185+
#
17186
]
18187
}
19188
]

‎.formatter.exs‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
# Used by "mix format"
22
[
3-
inputs: ["{mix,.formatter}.exs", "{config,lib,test,benchmarks}/**/*.{ex,exs}"]
3+
import_deps: [:nebulex],
4+
inputs: ["{mix,.formatter}.exs", "{config,lib,test,benchmarks}/**/*.{ex,exs}"],
5+
line_length: 100
46
]

‎.github/workflows/ci.yml‎

Lines changed: 43 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -9,30 +9,36 @@ on:
99
- main
1010

1111
jobs:
12-
nebulex_test:
12+
test:
1313
name: >-
14-
Nebulex.Adapters.Cachex Test (Elixir ${{ matrix.elixir }}
15-
/ OTP ${{ matrix.otp }} / OS ${{ matrix.os }})
14+
Nebulex.Adapters.Cachex Test (Elixir ${{ matrix.elixir }} /
15+
OTP ${{ matrix.otp }} /
16+
OS ${{ matrix.os }})
1617
runs-on: ${{ matrix.os }}
1718

1819
strategy:
1920
matrix:
2021
include:
21-
- elixir: 1.14.x
22-
otp: 25.x
22+
- elixir: 1.18.x
23+
otp: 27.x
2324
os: 'ubuntu-latest'
24-
style: true
25+
lint: true
2526
coverage: true
27+
sobelow: true
2628
dialyzer: true
27-
- elixir: 1.13.x
29+
- elixir: 1.17.x
30+
otp: 27.x
31+
os: 'ubuntu-latest'
32+
- elixir: 1.17.x
33+
otp: 25.x
34+
os: 'ubuntu-latest'
35+
- elixir: 1.14.x
2836
otp: 24.x
2937
os: 'ubuntu-latest'
30-
- elixir: 1.12.x
31-
otp: 23.x
32-
os: 'ubuntu-20.04'
3338

3439
env:
3540
GITHUB_TOKEN: '${{ secrets.GITHUB_TOKEN }}'
41+
CODECOV_TOKEN: '${{ secrets.CODECOV_TOKEN }}'
3642
MIX_ENV: test
3743
NEBULEX_PATH: nebulex
3844

@@ -53,6 +59,8 @@ jobs:
5359
key: >-
5460
${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-mix-${{
5561
hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
62+
restore-keys: |
63+
${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-mix-
5664
5765
- name: Cache _build
5866
uses: actions/cache@v3
@@ -61,6 +69,8 @@ jobs:
6169
key: >-
6270
${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-build-${{
6371
hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
72+
restore-keys: |
73+
${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-build-
6474
6575
- name: Install Nebulex
6676
run: mix nbx.setup
@@ -72,14 +82,22 @@ jobs:
7282
mix deps.get
7383
if: ${{ steps.mix-cache.outputs.cache-hit != 'true' }}
7484

75-
- name: Compile code
85+
- name: Compile deps
86+
run: mix deps.compile
87+
88+
- name: Check unused dependencies
89+
run: mix deps.unlock --check-unused
90+
if: ${{ matrix.lint }}
91+
92+
- name: Compile lint
7693
run: mix compile --warnings-as-errors
94+
if: ${{ matrix.lint }}
7795

7896
- name: Run style and code consistency checks
7997
run: |
8098
mix format --check-formatted
8199
mix credo --strict
82-
if: ${{ matrix.style }}
100+
if: ${{ matrix.lint }}
83101

84102
- name: Run tests
85103
run: |
@@ -90,15 +108,26 @@ jobs:
90108
- name: Run tests with coverage
91109
run: |
92110
epmd -daemon
93-
mix coveralls.github
111+
mix coveralls.json
112+
if: ${{ matrix.coverage }}
113+
114+
- name: Upload coverage reports to Codecov
115+
uses: codecov/codecov-action@v5
116+
with:
117+
fail_ci_if_error: true
118+
flags: unittests-elixir-${{ matrix.elixir }}-otp-${{ matrix.otp }}
94119
if: ${{ matrix.coverage }}
95120

121+
- name: Run sobelow
122+
run: mix sobelow --skip --exit Low
123+
if: ${{ matrix.sobelow }}
124+
96125
- name: Restore PLT Cache
97126
uses: actions/cache@v3
98127
id: plt-cache
99128
with:
100129
path: priv/plts
101-
key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plt-v2
130+
key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plt-v1
102131
if: ${{ matrix.dialyzer }}
103132

104133
- name: Create PLTs

‎.gitignore‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,5 @@ erl_crash.dump
3737
/priv
3838
.sobelow*
3939
/nebulex
40+
Elixir*
41+
*.coverdata

‎.tool-versions‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
elixir 1.18.3-otp-27
2+
erlang 27.3.2

LICENSE renamed to LICENSE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
MIT License
1+
# MIT License
22

33
Copyright (c) 2020 Carlos Andres Bolaños R.A.
44

0 commit comments

Comments
 (0)