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

tools: export the public API of the package #225

Merged
merged 4 commits into from
May 20, 2024

Conversation

vincenzopalazzo
Copy link
Owner

in python there is this concept of module defined by the __init__.py where you will go to define the public API.

In this way a user can import the API exposed by the package.

in python there is this concept of module defined by
the `__init__.py` where you will go to define the public
API.

In this way a user can import the API exposed by the package.

Signed-off-by: Vincenzo Palazzo <[email protected]>
@vincenzopalazzo vincenzopalazzo added the 👀 - seeking reviews This PR is seeking reviews label May 19, 2024
Copy link
Collaborator

@tareknaser tareknaser left a comment

Choose a reason for hiding this comment

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

Looks good to me

Signed-off-by: Vincenzo Palazzo <[email protected]>
@vincenzopalazzo
Copy link
Owner Author

Push the changes requested by @tareknaser so merging this

@vincenzopalazzo vincenzopalazzo merged commit 45dc214 into main May 20, 2024
3 of 5 checks passed
@vincenzopalazzo vincenzopalazzo deleted the macros/tools-rpclib branch May 20, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌎 - language-bindings Related to the language bindings 👀 - seeking reviews This PR is seeking reviews
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants