chore: max-params warning for src/sdk/sdk.ts

[tamirazrab]

Description

Resolve max-params warning for src/sdk/sdk.ts
...

End to End Test:
(See Pepr Excellent Examples)

Related Issue

Relates to #1700

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Other (security config, docs update, etc)

Checklist before merging

• Unit, Journey, E2E Tests, docs, adr added or updated as needed
• Contributor Guide Steps followed

[cmwylie19]

Code looks good but this is user facing breaking change

https://docs.pepr.dev/main/user-guide/sdk/#writeevent

We would need to deprecate before making this change. We will chat as a team and give you some feedback. Thanks for the PR

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.12%. Comparing base (129376b) to head (d77ff1a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1728   +/-   ##
=======================================
  Coverage   82.11%   82.12%           
=======================================
  Files          52       52           
  Lines        2214     2215    +1     
  Branches      459      469   +10     
=======================================
+ Hits         1818     1819    +1     
- Misses        367      394   +27     
+ Partials       29        2   -27     

Files with missing lines	Coverage Δ
src/sdk/sdk.ts	100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

[samayer12]

As an update, we're still thinking about how we want to approach the underlying program structure that caused this warning. In general, we prefer to avoid resolving max-params by creating an object that contains N properties. However, if that's the best solution, we can work with it.

[cmwylie19]

lgtm, we will leave a message in our release about this change

[samayer12]

We have a plan to communicate this breaking change to users, good to go!

[cmwylie19]

Looks like some commits are not signed. Can you sign them or open up a new PR with these changes?