-
Notifications
You must be signed in to change notification settings - Fork 413
/
unused_path_operation.ex
45 lines (35 loc) · 1.01 KB
/
unused_path_operation.ex
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
defmodule Credo.Check.Warning.UnusedPathOperation do
use Credo.Check,
base_priority: :high,
explanations: [
check: """
The result of a call to the Path module's functions has to be used.
While this is correct ...
def read_from_cwd(filename) do
filename = Path.join(cwd, filename)
File.read(filename)
end
... we forgot to save the result in this example:
def read_from_cwd(filename) do
Path.join(cwd, filename)
File.read(filename)
end
Path operations never work on the variable you pass in, but return a new
variable which has to be used somehow.
"""
]
alias Credo.Check.Warning.UnusedOperation
@checked_module :Path
@funs_with_return_value nil
@doc false
@impl true
def run(%SourceFile{} = source_file, params) do
UnusedOperation.run(
source_file,
params,
@checked_module,
@funs_with_return_value,
&format_issue/2
)
end
end