Linting as a separate stage
This commit is contained in:
parent
a0518a4ee1
commit
df31ec0d52
|
@ -28,6 +28,7 @@ cache: &global_cache_policy
|
||||||
stages:
|
stages:
|
||||||
- check-changelog
|
- check-changelog
|
||||||
- build
|
- build
|
||||||
|
- lint
|
||||||
- test
|
- test
|
||||||
- benchmark
|
- benchmark
|
||||||
- deploy
|
- deploy
|
||||||
|
@ -185,10 +186,10 @@ unit-testing-rum:
|
||||||
- "mix ecto.migrate --migrations-path priv/repo/optional_migrations/rum_indexing/"
|
- "mix ecto.migrate --migrations-path priv/repo/optional_migrations/rum_indexing/"
|
||||||
- mix test --preload-modules
|
- mix test --preload-modules
|
||||||
|
|
||||||
lint:
|
formatting:
|
||||||
extends: .build_changes_policy
|
extends: .build_changes_policy
|
||||||
image: ¤t_elixir elixir:1.13-alpine
|
image: ¤t_elixir elixir:1.13-alpine
|
||||||
stage: test
|
stage: lint
|
||||||
cache: *testing_cache_policy
|
cache: *testing_cache_policy
|
||||||
before_script: ¤t_bfr_script
|
before_script: ¤t_bfr_script
|
||||||
- apk update
|
- apk update
|
||||||
|
@ -203,7 +204,7 @@ analysis:
|
||||||
extends:
|
extends:
|
||||||
- .build_changes_policy
|
- .build_changes_policy
|
||||||
- .using-ci-base
|
- .using-ci-base
|
||||||
stage: test
|
stage: lint
|
||||||
cache: *testing_cache_policy
|
cache: *testing_cache_policy
|
||||||
script:
|
script:
|
||||||
- mix credo --strict --only=warnings,todo,fixme,consistency,readability
|
- mix credo --strict --only=warnings,todo,fixme,consistency,readability
|
||||||
|
@ -211,7 +212,7 @@ analysis:
|
||||||
cycles:
|
cycles:
|
||||||
extends: .build_changes_policy
|
extends: .build_changes_policy
|
||||||
image: *current_elixir
|
image: *current_elixir
|
||||||
stage: test
|
stage: lint
|
||||||
cache: {}
|
cache: {}
|
||||||
before_script: *current_bfr_script
|
before_script: *current_bfr_script
|
||||||
script:
|
script:
|
||||||
|
|
Loading…
Reference in New Issue