summaryrefslogtreecommitdiff
path: root/danger/prettier/Dangerfile
blob: 86f9f6af475a9852f226082d250a5957a5d3d311 (plain)
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
# frozen_string_literal: true

def get_prettier_files(files)
  files.select do |file|
    file.end_with?('.js', '.scss', '.vue')
  end
end

prettier_candidates = get_prettier_files(git.added_files + git.modified_files)

return if prettier_candidates.empty?

unpretty = `node_modules/prettier/bin-prettier.js --list-different #{prettier_candidates.join(" ")}`
             .split(/$/)
             .map(&:strip)
             .reject(&:empty?)

return if unpretty.empty?

warn 'This merge request changed frontend files without pretty printing them.'

markdown(<<~MARKDOWN)
  ## Pretty print Frontend files

  The following files should have been pretty printed with `prettier`:

  * #{unpretty.map { |path| "`#{path}`" }.join("\n* ")}

  Please run

  ```
  node_modules/.bin/prettier --write \\
  #{unpretty.map { |path| "  '#{path}'" }.join(" \\\n")}
  ```

  Also consider auto-formatting [on-save].

  [on-save]: https://docs.gitlab.com/ee/development/new_fe_guide/style/prettier.html
MARKDOWN