etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
184 stars 28 forks source link

Warnings lors du mix check_all #3977

Open vdegove opened 4 weeks ago

vdegove commented 4 weeks ago

Lorsque je fais mix test après un mix clean, je n’ai pas de warnings, si j’enchaîne avec mix check_all j’en ai. L’appli est bizarrement recompilée lors du mix check_all alors qu’aucun fichier n’est touché. Une fois que j’ai eu des warnings, en refaisant mix test ou mix check_all, j’ai toujours des warnings jusqu’au mix clean suivant.

[vincent@waluigi]~/Code/transport-site% mix test
==> shared
Compiling 26 files (.ex)
Generating Transport.Cldr for 3 locales named [:en, :fr, :und] with a default locale named :fr
Generated shared app
==> datagouvfr
Compiling 16 files (.ex)
Generated datagouvfr app
==> unlock
Compiling 15 files (.ex)
Generated unlock app
==> gbfs
Compiling 17 files (.ex)
Generated gbfs app
==> transport
Compiling 304 files (.ex)
Generated transport app

14:08:29.510 [info] Migrations already up
==> shared
...........................................................................
Finished in 0.3 seconds (0.2s async, 0.1s sync)
21 doctests, 54 tests, 0 failures
[…]

[vincent@waluigi]~/Code/transport-site% mix check_all  

> linter:sass
> stylelint --config '../../../.stylelintrc.json' 'stylesheets/**/*.scss'

> linter:ecma
> eslint -c .eslintrc.json --resolve-plugins-relative-to ./apps/transport/client/node_modules --ext .js .

Checking 788 source files (this might take a while) ...

Please report incorrect results: https://github.com/rrrene/credo/issues

Analysis took 2.6 seconds (0.4s to load, 2.2s running 67 checks on 788 files)
4428 mods/funs, found no issues.

Use `mix credo explain` to explain issues, `mix credo --help` for options.
==> shared
Compiling 26 files (.ex)
Generating Transport.Cldr for 3 locales named [:en, :fr, :und] with a default locale named :fr
==> unlock
Compiling 2 files (.ex)
==> gbfs
Compiling 2 files (.ex)
==> transport
Compiling 22 files (.ex)
==> datagouvfr
Compiling 16 files (.ex)
    warning: redefining module Datagouvfr.Client.Datasets (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/datagouvfr/ebin/Elixir.Datagouvfr.Client.Datasets.beam)
    │
  1 │ defmodule Datagouvfr.Client.Datasets do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/datagouvfr/client/datasets.ex:1: Datagouvfr.Client.Datasets (module)

    warning: redefining module Datagouvfr.Gettext (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/datagouvfr/ebin/Elixir.Datagouvfr.Gettext.beam)
    │
  1 │ defmodule Datagouvfr.Gettext do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/gettext.ex:1: Datagouvfr.Gettext (module)

    warning: the Vex.Blank protocol has already been consolidated, an implementation for Datagouvfr.Client.Datasets has no effect. If you want to implement protocols after compilation or during tests, check the "Consolidation" section in the Protocol module documentation
    │
  1 │ defmodule Datagouvfr.Client.Datasets do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/datagouvfr/client/datasets.ex:1: Datagouvfr.Client.Datasets (module)

    warning: the Vex.Extract protocol has already been consolidated, an implementation for Datagouvfr.Client.Datasets has no effect. If you want to implement protocols after compilation or during tests, check the "Consolidation" section in the Protocol module documentation
    │
  1 │ defmodule Datagouvfr.Client.Datasets do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/datagouvfr/client/datasets.ex:1: Datagouvfr.Client.Datasets (module)

==> unlock
Compiling 15 files (.ex)
    warning: redefining module Unlock.Endpoint (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.Endpoint.beam)
    │
  3 │ defmodule Unlock.Endpoint do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/endpoint.ex:3: Unlock.Endpoint (module)

    warning: redefining module Unlock.GunzipTools (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.GunzipTools.beam)
    │
  1 │ defmodule Unlock.GunzipTools do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/gunzip_tools.ex:1: Unlock.GunzipTools (module)

    warning: redefining module Unlock.Router (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.Router.beam)
    │
  1 │ defmodule Unlock.Router do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/router.ex:1: Unlock.Router (module)

    warning: redefining module Unlock.Telemetry (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.Telemetry.beam)
    │
  1 │ defmodule Unlock.Telemetry do
    │ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/telemetry.ex:1: Unlock.Telemetry (module)

    warning: redefining module Unlock.Config.Item.Generic.HTTP (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.Config.Item.Generic.HTTP.beam)
    │
  7 │   defmodule Item.Generic.HTTP do
    │   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/config.ex:7: Unlock.Config.Item.Generic.HTTP (module)

    warning: redefining module Unlock.HTTP.Response (current version loaded from /home/vincent/Code/transport-site/_build/test/lib/unlock/ebin/Elixir.Unlock.HTTP.Response.beam)
    │
  8 │   defmodule Response do
    │   ~~~~~~~~~~~~~~~~~~~~~
    │
    └─ lib/http_client.ex:8: Unlock.HTTP.Response (module)
[…]
ptitfred commented 4 weeks ago

Je suspecte mix format --check-formatted d'interférer.