Skip to content

feat: add coming-from parameter - #837

Merged
obostjancic merged 5 commits into
masterfrom
ogi/feat/partner-param
Mar 5, 2025
Merged

obostjancic merged 5 commits into
masterfrom
ogi/feat/partner-param

Conversation

@obostjancic

Copy link
Copy Markdown
Member

@github-actions

github-actions Bot commented Mar 4, 2025 •

Copy link
Copy Markdown
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against be6c1e2

@codecov

codecov Bot commented Mar 4, 2025 •

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 51.15%. Comparing base (2d006c9) to head (be6c1e2).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/utils/clack-utils.ts 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #837      +/-   ##
==========================================
- Coverage   51.15%   51.15%   -0.01%     
==========================================
  Files          53       53              
  Lines        3517     3519       +2     
  Branches      825      827       +2     
==========================================
+ Hits         1799     1800       +1     
- Misses       1591     1718     +127     
+ Partials      127        1     -126     
Flag Coverage Δ
e2e-tests 80.85% <ø> (ø)
unit-tests 49.91% <66.66%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@obostjancic
obostjancic requested a review from Lms24 March 5, 2025 09:29
@obostjancic
obostjancic marked this pull request as ready for review March 5, 2025 09:29

@lforst lforst left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code-wise lgtm. However "partner" sounds extremely weird from a user-perspective as a CLI arg. I would name it something like --coming-from.

@Lms24 Lms24 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please also update the readme with the new arg, thx!

@obostjancic obostjancic changed the title feat: add partner parameter feat: add coming-from parameter Mar 5, 2025
@obostjancic
obostjancic merged commit 273513d into master Mar 5, 2025
@obostjancic
obostjancic deleted the ogi/feat/partner-param branch March 5, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants