Skip to content

Conversation

@Anaethelion
Copy link
Contributor

This adds a constructor for the BaseClient without an API attached.

Easing up usage by passing the client into the endpoints, allowing for a slimmer compiled binary.

@Anaethelion Anaethelion merged commit 1461d61 into main Apr 16, 2025
16 of 17 checks passed
@Anaethelion Anaethelion deleted the new_base_client branch April 16, 2025 17:23
github-actions bot pushed a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct
github-actions bot pushed a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct
github-actions bot pushed a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct
Anaethelion added a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct

Co-authored-by: Laurent Saint-Félix <[email protected]>
Anaethelion added a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct

Co-authored-by: Laurent Saint-Félix <[email protected]>
Anaethelion added a commit that referenced this pull request Apr 16, 2025
* add constructor for BaseClient

* expose instrument for esapi to allow injection via struct

* use exposed instrument for esapi to allow injection via struct

Co-authored-by: Laurent Saint-Félix <[email protected]>
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