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.

Sign up for GitHub

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

Jump to bottom

Internal: use NewType for ID objects #10

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Internal: use NewType for ID objects #10

merged 1 commit into from
Sep 25, 2023

Commits on Sep 22, 2023

  1. Internal: use NewType for ID objects 

    Problem: ID objects (request ID, execution ID, nonce) all use raw
string/int objects. This could lead to using the wrong object in the
wrong place without noticing.

Solution: defined dedicated types for each object type.
    @odesenfans
    odesenfans committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    7d21d73 View commit details
    Browse the repository at this point in the history