sentry-cli icon indicating copy to clipboard operation
sentry-cli copied to clipboard

API refactor: Split data types into separate submodules

Open szokeasaurusrex opened this issue 1 year ago • 4 comments

Data types are types which exist primarily for serializing/deserializing request/response data. Examples include User and ReleaseStatus.

szokeasaurusrex avatar Jun 03 '24 11:06 szokeasaurusrex

Or, perhaps it makes more sense to move this into sentry-rust

szokeasaurusrex avatar Jun 05 '24 19:06 szokeasaurusrex

Maybe get @Swatinem or @loewenheim's opinion on that

stephanie-anderson avatar Jun 06 '24 11:06 stephanie-anderson

Also discussed with @elramen recently that IMO we shouldn’t bloat the SDK with types and definitions that are not directly useful for the SDK itself.

Swatinem avatar Jun 07 '24 08:06 Swatinem

@Swatinem I was considering that we might implement this change as a new feature under sentry-types, so as not to increase binary size when not needed. The benefit would be that we could reuse this code in other Rust packages.

However, you have a fair point – the code would be, at least initially, only used in the CLI, so probably makes more sense to keep the code in the CLI. We can always reevaluate later.

Another idea I had would be perhaps that we could work towards splitting off reusable CLI code into a separate library (or perhaps, multiple libraries), which the CLI would use, and which we could reuse elsewhere. But, this would probably be a more long-term initiative

szokeasaurusrex avatar Jun 07 '24 15:06 szokeasaurusrex

See comment on #2008

szokeasaurusrex avatar Dec 10 '24 10:12 szokeasaurusrex