Fix elixir 1.17 and migration lock warnings #810

Merged

floatingghost merged 2 commits from Oneric/akkoma:ex1.17-warnings into develop

2024-06-23 02:18:42 +00:00

Oneric commented

2024-06-19 00:23:36 +00:00

Member

Remaining deprecation warnings are in external deps and the captcha dep from pleroma

tested MIX_ENV=test mix ecto.reset, mix test and mix compile

EDIT: for the deprecation warning from elixir-captcha which pops up on every mix invocation, i sent the trivial fix to Pleroma’s upstream, not sure if/when this get’s merged though or if forking it to akkoma.dev would be better: https://git.pleroma.social/pleroma/elixir-libraries/elixir-captcha/-/merge_requests/6

Remaining deprecation warnings are in external deps and the captcha dep from pleroma tested `MIX_ENV=test mix ecto.reset`, `mix test` and `mix compile` **EDIT**: for the deprecation warning from elixir-captcha which pops up on every `mix` invocation, i sent the trivial fix to Pleroma’s upstream, not sure if/when this get’s merged though or if forking it to akkoma.dev would be better: https://git.pleroma.social/pleroma/elixir-libraries/elixir-captcha/-/merge_requests/6

Oneric added 2 commits 2024-06-19 00:23:36 +00:00

cosmetic: fix elixir 1.17 compiler warnings in main application c3069b9478

Fix concurrent index creation warnings

ci/woodpecker/pr/build-amd64 Pipeline is pending

Details

ci/woodpecker/pr/build-arm64 Pipeline is pending

Details

ci/woodpecker/pr/docs Pipeline is pending

Details

ci/woodpecker/pr/lint Pipeline is pending

Details

ci/woodpecker/pr/test Pipeline is pending

Details

b9391adecf

Since those old migrations will now most likely only run during db init,
there’s not much point in running them in the background concurrently
anyway, so just drop the cncurrent setting rather than disabling
migration locks.

Oneric force-pushed ex1.17-warnings from b9391adecf to 1a4238bf98

2024-06-19 00:25:48 +00:00

Compare