-
Notifications
You must be signed in to change notification settings - Fork 61
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
Update trestle project structure to better define the SDK APIs #1713
Comments
@degenaro @vikas-agarwal76 @butler54 I wanted to start collaboration on the structure here so I thought the most straightforward way might be to separate the existing sub-packages into categories. Let me know what you think or if you would rather look at a draft pull request. To summarize an idea that we could use as a jumping off point:
|
@jpower432 Seems like a good first swag. Maybe tests should be likewise structured? |
@vikas-agarwal76 @mrgadgil @AleJo2995 @butler54 @degenaro Following up from the review today. I am looking for feedback on this structure and the approach to implementing the change. Proposing the following approach:
I also wanted to ask for everyone's thoughts on versioning. Since the public API is not fully defined yet, do we need to bump a major version if this results in no change to the trestle SDK end user? |
Issue description / feature objectives
To support the
trestle
SDK user, create well-defined APIs so users can understand the SDK functionality what API signatures cannot be changed without breaking backward compatibility.Caveats / Assumptions
Completion Criteria
list-utils
)trestle
SDKProposed Child Issues
Related to #924
The text was updated successfully, but these errors were encountered: