-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Liste des warnings Elixir #1691
Labels
dette technique
Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité
Comments
thbar
added
the
dette technique
Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité
label
Jul 1, 2021
thbar
added a commit
that referenced
this issue
Jul 5, 2021
See https://ninenines.eu/docs/en/cowboy/2.9/guide/migrating_from_2.8/ This upgrades cowboy, cowlib, and also ranch.
This was referenced Jul 6, 2021
thbar
added a commit
that referenced
this issue
Jul 9, 2021
* Bump up Elixir & OTP to latest * Revert to version available as hex.pm image * Bump up base image + OTP & Elixir Hex.pm has published a new image, which has allowed me to release (https://github.com/etalab/transport-ops/releases/tag/elixir-1.12.1-erlang-24.0.3-alpine-3.13.3) a compatible image. * Fix base image reference * Trigger CI build * Update credo to latest and fix credo blockers * Fix postgrex-related Dialyzer warnings See elixir-ecto/postgrex#549 * Apply mix format * Remove warning during tests * Upgrade cowboy & ranch for OTP 24 compatibility (#1691) See https://ninenines.eu/docs/en/cowboy/2.9/guide/migrating_from_2.8/ This upgrades cowboy, cowlib, and also ranch. * Upgrade mochiweb to latest (OTP 24 compat) I had for now to work-around the lack of official hex.pm release (mochi/mochiweb#233). * Update Vex to master for Elixir 1.11 compat I also removed vex dependency for apps which did not depend on vex. * Bump up Elixir to 1.12.2 * Backport coveralls fixes from #1446 * Bump up coveralls * Fix credo warning & use fetch_env! instead of get for config * Skip credo issue for now * Fix regression after refactoring
Je clôture pour l'instant ; ce ticket m'avait aidé pour #1336 et associés. Il existe toujours de nombreux warnings, mais on fera un traitement en temps et en heure. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
dette technique
Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité
Je fais une méta-issue relative à #1683, mais pas seulement, où je vais lister les warnings individuellement ici. Ca permettra de faire un suivi car il y en a beaucoup, et aussi de faciliter le traitement ou individuellement, ou par groupes, via différentes PR.
no configuration found for otp_app :unlock and module Unlock.Endpoint
Mise à jour Elixir 1.10 -> 1.12 et OTP 23 -> 24 #1683warning: :pool_timeout option no longer has an effect and has been replaced with an improved queuing system. See "Queue config" in DBConnection.start_link/2 documentation for more information.
(vu en production via iex)===> Compiling ranch src/ranch_ssl.erl:142:7: Warning: ssl:ssl_accept/3 is removed; use ssl_handshake/1,2,3 instead
-> il faut mettreranch
à jour Release Ranch 1.8 that is compatible with OTP-24 ninenines/ranch#314 (traité dans Mise à jour Elixir 1.10 -> 1.12 et OTP 23 -> 24 #1683)===> Compiling mochiweb src/mochiweb_session.erl:145:13: Warning: crypto:block_encrypt/4 is removed; use crypto:crypto_one_time/5, crypto:crypto_one_time_aead/6,7 or crypto:crypto_(dyn_iv)?_init + crypto:crypto_(dyn_iv)?_update + crypto:crypto_final instead
, voirmix deps.tree | grep mochi -C 1
,mochiweb
est une dépendance dehtml_sanitize_ex
==> elixir_uuid Compiling 1 file (.ex) warning: :crypto.hash/2 defined in application :crypto is used by the current application but the current application does not depend on :crypto. To fix this, you must do one of:
-> il y aurait un fix ici updating dependencies, addressing compiler warnings zyro/elixir-uuid#53 mais non mergé, et le mainteneur ne répond pas. Cette dépendance est pullée parexshape
, je dois vérifier si on peut la supprimer potentiellement -> EDIT: oui je la supprime ici Suppression de la librairie exshape (et elixir_uuid) #1695==> vex Compiling 25 files (.ex) warning: do not use "length(errors) > 0" to check if a list is not empty since length always traverses the whole list. Prefer to pattern match on a non-empty list, such as [_ | _], or use "errors != []" as a guard lib/vex.ex:15 warning: module attribute @only was set but never used lib/vex/blank.ex:2
voir New release to remove compilation warnings? CargoSense/vex#68==> tzdata warning: :random.uniform/1 is deprecated. Use the 'rand' module instead lib/tzdata/data_loader.ex:17: Tzdata.DataLoader.download_new/1
et beaucoup d'autres Mise à jour timex & tzdata pour Elixir 1.12 #1700 Fix warnings in Elixir 1.11 lau/tzdata#106Logs actuels
Au commit 4b3c89c, on avait encore les logs suivants:
Click to expand!
The text was updated successfully, but these errors were encountered: