Skip to content

Update for modern Elixir (1.12+)#1

Merged
E14 merged 1 commit into
masterfrom
up
Jan 5, 2026
Merged

Update for modern Elixir (1.12+)#1
E14 merged 1 commit into
masterfrom
up

Conversation

@E14

@E14 E14 commented Jan 3, 2026

Copy link
Copy Markdown
Owner
  • Update dependencies
  • config/config.exs: Use Config instead of Mix.Config
  • lib/trot.ex: Fix Trot.parse_query_string/1
  • lib/trot/router.ex: Use Application.compile_env/3 instead of Application.get_env/3
  • test/trot/route_test.exs: Use inspect/1 in unit test as well to avoid unstable test due to changing rendering

@E14 E14 force-pushed the up branch 3 times, most recently from 1f7db0b to ba4f762 Compare January 3, 2026 22:53
@E14 E14 changed the title Update for modern Elixir (1.11+) Update for modern Elixir (1.12+) Jan 5, 2026
@E14 E14 force-pushed the up branch 7 times, most recently from 495a064 to 0b23f35 Compare January 5, 2026 14:13
Update dependencies for Elixir 1.12+
- Increase minimum versions for poison, calliope, plug_heartbeat, credo, slime
- Increase minimum ex_doc version to 0.33 - removes requirement for earmark, requires Elixir 1.11+
- Removed :applications, added :extra_applications instead

Update
- config/config.exs: Use Config instead of Mix.Config
- lib/trot.ex: Fix Trot.parse_query_string/1 for Elixir 1.12+
- lib/trot/router.ex: Use Application.compile_env/3 instead of Application.get_env/3
- test/trot/route_test.exs: Use inspect/1 in unit test as well to avoid unstable test due to changing rendering
- test/trot/live_reload_test.exs: Create temporary file instead of touching trot.ex

Actions:
- Fix incorrect cache hits in workflows by including Elixir version in cache key
- Use `> 0` instead  of `latest` for elixir and  otp versions
@E14 E14 merged commit f6471f6 into master Jan 5, 2026
1 of 2 checks passed
@E14 E14 deleted the up branch January 5, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant