Skip to content

Make JSON library configurable, default to Jason#36

Merged
halfdan merged 7 commits intomasterfrom
fb/scl52
Mar 3, 2025
Merged

Make JSON library configurable, default to Jason#36
halfdan merged 7 commits intomasterfrom
fb/scl52

Conversation

@halfdan
Copy link
Copy Markdown
Contributor

@halfdan halfdan commented Feb 24, 2025

This PR:

  • Updates dependencies
  • Swaps out Poison for Jason
  • Makes the JSON library configurable (inspired by Phoenix)
  • Adds a build matrix to test against multiple Ex/OTP pairs (taken from Styler)

@halfdan halfdan changed the title Add build matrix and separate compile step Make JSON library configurable, default to Jason Feb 24, 2025
@halfdan halfdan requested a review from milmazz February 24, 2025 09:26
@halfdan halfdan force-pushed the fb/scl52 branch 2 times, most recently from ed8a2bb to 4112f11 Compare February 24, 2025 09:37
Comment thread .github/workflows/elixir.yml Outdated
Comment thread .github/workflows/elixir.yml
halfdan and others added 2 commits February 25, 2025 20:34
Co-authored-by: Milton Mazzarri <[email protected]>
Co-authored-by: Milton Mazzarri <[email protected]>
Copy link
Copy Markdown
Contributor

@milmazz milmazz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The additions already caught an unused dependency 👍

@halfdan halfdan merged commit b906022 into master Mar 3, 2025
@halfdan halfdan deleted the fb/scl52 branch March 3, 2025 08:37
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.

2 participants