Skip to content

Conversation

IvanHristov98
Copy link
Contributor

PR Details

Implements the UNIQUE function in excel.

Description

Implements the UNIQUE function in excel.
https://support.microsoft.com/en-us/office/unique-function-c5ab87fd-30a3-4ce9-9d1a-40204fb85e1e

Related Issue

#2165

Motivation and Context

So that I can use UNIQUE in excel. It is a very useful function.

How Has This Been Tested

Unit tests were added. The test cases were derived with the help of the official MS docs and I checked them.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@IvanHristov98
Copy link
Contributor Author

Hmm. This adds some other stuff that we want to contribute from our fork. Will fix it to leave out only the UNIQUE function.

@IvanHristov98 IvanHristov98 changed the title Add unique function DRAFT Add unique function Jul 3, 2025
Copy link

codecov bot commented Jul 3, 2025

Codecov Report

Attention: Patch coverage is 90.65421% with 10 lines in your changes missing coverage. Please review.

Project coverage is 99.20%. Comparing base (55e152f) to head (b156de9).
Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
calc.go 90.65% 7 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2166      +/-   ##
==========================================
- Coverage   99.23%   99.20%   -0.04%     
==========================================
  Files          32       32              
  Lines       30379    30486     +107     
==========================================
+ Hits        30148    30245      +97     
- Misses        153      160       +7     
- Partials       78       81       +3     
Flag Coverage Δ
unittests 99.20% <90.65%> (-0.04%) ⬇️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@xuri xuri added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Jul 3, 2025
* Implement UNIQUE function

* Reduce memory footprint

---------

Co-authored-by: Ivan Hristov <[email protected]>
@IvanHristov98 IvanHristov98 force-pushed the add-unique-function branch from e2d0a49 to b156de9 Compare July 4, 2025 12:02
@IvanHristov98 IvanHristov98 changed the title DRAFT Add unique function Add unique function Jul 4, 2025
@IvanHristov98
Copy link
Contributor Author

I removed the redundant commits. I also performed a refactoring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants