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

refactor: move type matching to own function #3656

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Oct 11, 2023

Follow-up for #3653

  • Move matching to own function
  • Tests
  • Additional check for the bonus point if also the intersection has the same length

@kesselb kesselb added enhancement New feature or request 3. to review Waiting for reviews labels Oct 11, 2023
@kesselb kesselb self-assigned this Oct 11, 2023
tests/javascript/utils/matchTypes.test.ts Outdated Show resolved Hide resolved
tests/javascript/utils/matchTypes.test.ts Outdated Show resolved Hide resolved
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb force-pushed the bug/3651/update-matching-score-2 branch from 9146c32 to 1910947 Compare October 11, 2023 14:53
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (106bd3b) 1.80% compared to head (1910947) 2.07%.

Additional details and impacted files
@@            Coverage Diff             @@
##              main   #3656      +/-   ##
==========================================
+ Coverage     1.80%   2.07%   +0.26%     
==========================================
  Files          113      89      -24     
  Lines         6136    5343     -793     
  Branches      1492    1491       -1     
==========================================
  Hits           111     111              
+ Misses        5907    5114     -793     
  Partials       118     118              
Files Coverage Δ
...mponents/ContactDetails/ContactDetailsProperty.vue 0.00% <ø> (ø)

... and 24 files with indirect coverage changes

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

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

Fancy typescript code ✨

@st3iny st3iny merged commit 8d06b84 into main Oct 17, 2023
@st3iny st3iny deleted the bug/3651/update-matching-score-2 branch October 17, 2023 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants