Skip to content
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

chore(deps): update dependency protox to v2 #293

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 15, 2025

This PR contains the following updates:

Package Type Update Change
protox (source) prod major ~> 1.7 -> ~> 2.0

Release Notes

ahamez/protox (protox)

v2.0.0

Compare Source

Added
  • schema/0 to access the underlying definition of a message.
Changed
  • Minimal supported Elixir version is now 1.15.
  • BREAKING CHANGE: encoding functions now return the size alongside iodata.
    Previously, one had to use :binary.list_to_bin/1 to flatten the iodata and then use byte_size, defeating the purpose of having an iodata. Furthermore, it provides a significant performance boost.
  • BREAKING CHANGE: Protox.decode! and Protox.decode no longer raise or return an error if a message don't have all required fields set. We should let the user decide whether it's a problem.
  • BREAKING CHANGE: Rename InvalidFieldAttribute to InvalidFieldAttributeError for consistency.
Removed
  • BREAKING CHANGE: Drop support of JSON encoding and decoding.
    The implementation was half-baked, it's preferable to have the library focused. Even Google doesn't recommend using JSON: "In short, there are many good reasons why Google prefers to use the standard wire format for virtually everything rather than ProtoJSON format".
  • BREAKING CHANGE: Remove :path option in favor of the already existing :paths option, which covers the same functionality.
  • BREAKING CHANGE: Remove :keep_unknown_fields option (the corresponding --keep-unknown-fields option for the mix task protox.generate is also removed). They are now always kept.
    It added complexity to the generation logic while not providing any value as one can simply ignore those fields and as it's mandatory to parse them correctly.
  • BREAKING CHANGE: Remove generated defs/0 (functionality remains available through schema/0).
  • BREAKING CHANGE: Remove generated field_def/1 (functionality remains available through schema/0).
  • BREAKING CHANGE: Remove generated file_options/0 (functionality remains available through schema/0).
  • BREAKING CHANGE: Remove generated required_fields/0 (functionality remains available through schema/0).
  • BREAKING CHANGE: Remove generated syntax/0 (functionality remains available through schema/0).
  • BREAKING CHANGE: Remove generated encode/1 from strings for enums.
  • BREAKING CHANGE: Remove Protox.Encode.encode/1 and Protox.Encode.encode!/1 (functionality remains available in generated modules and in Protox).

v1.7.8

Compare Source

Fixed
  • Fix warnings of unknown JSON modules when a JSON library is not installed

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot force-pushed the renovate/protox-2.x branch 2 times, most recently from 4bbe799 to 297c7ff Compare March 21, 2025 17:05
@renovate renovate bot force-pushed the renovate/protox-2.x branch 3 times, most recently from 662d440 to fa3d1b6 Compare March 28, 2025 14:18
@renovate renovate bot force-pushed the renovate/protox-2.x branch 3 times, most recently from e5f6120 to 41c232f Compare April 5, 2025 04:33
@renovate renovate bot force-pushed the renovate/protox-2.x branch from 41c232f to a836e1c Compare April 11, 2025 21:12
@codecov-commenter
Copy link

codecov-commenter commented Apr 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (60d9156) to head (5d01f1b).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #293   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          200       200           
=========================================
  Hits           200       200           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@renovate renovate bot force-pushed the renovate/protox-2.x branch from a836e1c to 5d01f1b Compare April 12, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant