Skip to content

TMP PR#1559

Closed
sequba wants to merge 16 commits into
masterfrom
release/3.1.0
Closed

TMP PR#1559
sequba wants to merge 16 commits into
masterfrom
release/3.1.0

Conversation

@sequba

@sequba sequba commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Context

How did you test your changes?

Types of changes

  • Breaking change (a fix or a feature because of which an existing functionality doesn't work as expected anymore)
  • New feature or improvement (a non-breaking change that adds functionality)
  • Bug fix (a non-breaking change that fixes an issue)
  • Additional language file, or a change to an existing language file (translations)
  • Change to the documentation

Related issues:

  1. Fixes #...

Checklist:

  • I have reviewed the guidelines about Contributing to HyperFormula and I confirm that my code follows the code style of this project.
  • I have signed the Contributor License Agreement.
  • My change is compliant with the OpenDocument standard.
  • My change is compatible with Microsoft Excel.
  • My change is compatible with Google Sheets.
  • I described my changes in the CHANGELOG.md file.
  • My changes require a documentation update.
  • My changes require a migration guide.

sequba and others added 16 commits August 11, 2025 10:53
* Add test/compatibility subdirectory

* Install exceljs lib

* Compare evaluation results

* Use epsilon to compare numbers

* Handle cell errors

* Add script that runs all the test files in a loop

* Handle null-like values in xlsx files

* Update file-import guide in the docs to handle empty cells correctly

* Add test:compatibility to npm run test script

* Add test/compatibility/README.md file
… the Vue troubleshooting section (#1549)

Mention markRaw function in the Vue troubleshooting section
…1535)

* Add unit test to make sure issue 1297 will not be introduced again in the future

* Disable 'Unchanged files with check annotations' section in PR

* Revert last commit

* Revert change made by mistake

* Add jsdoc comment

* Fix type error
… the provided address (#1548)

* Add unit tests that reproduce the issue

* Add changelog entry

* Make OFFSET read the sheet ref in the provided address

* Adjust eslint rules

* Fix TS type warning

* Fix eslint warning
Docs: describe how HyperFormula determines the cell value type
Docs: Describe units supported by DATEDIF function
…sic-operations guide (#1557)

Add links to API reference to the methods description in the basic-operations guide
#1558)

* Deprecate FunctionMetadata.arrayFunction in favor of enableArrayArithmeticForArguments

* Add unit tests for depracated function metadata

* Depracate arraySizeMethod param in favor of sizeOfResultArrayMethod

* Add changelog entry

* Tweak the docs
@codecov

codecov Bot commented Oct 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@b72f797). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1559   +/-   ##
=========================================
  Coverage          ?   97.22%           
=========================================
  Files             ?      169           
  Lines             ?    14535           
  Branches          ?     3173           
=========================================
  Hits              ?    14132           
  Misses            ?      396           
  Partials          ?        7           
Files with missing lines Coverage Δ
src/AbsoluteCellRange.ts 94.59% <100.00%> (ø)
src/ArraySize.ts 98.21% <100.00%> (ø)
src/HyperFormula.ts 99.73% <ø> (ø)
src/interpreter/FunctionRegistry.ts 99.37% <100.00%> (ø)
src/interpreter/plugin/ArrayPlugin.ts 98.64% <100.00%> (ø)
src/interpreter/plugin/FunctionPlugin.ts 99.02% <ø> (ø)
src/interpreter/plugin/LookupPlugin.ts 97.18% <100.00%> (ø)
src/interpreter/plugin/MatrixPlugin.ts 97.88% <100.00%> (ø)
src/parser/FormulaParser.ts 97.61% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Oct 8, 2025

Copy link
Copy Markdown

Performance comparison of head (c7ec7e4) vs base (b72f797)

                                     testName |   base |   head | change
------------------------------------------------------------------------
                                      Sheet A | 496.95 | 501.41 | +0.90%
                                      Sheet B | 163.16 | 167.42 | +2.61%
                                      Sheet T | 143.47 | 145.11 | +1.14%
                                Column ranges | 496.25 |  501.7 | +1.10%
Sheet A:  change value, add/remove row/column |  13.83 |  13.72 | -0.80%
 Sheet B: change value, add/remove row/column | 131.32 | 130.99 | -0.25%
                   Column ranges - add column |  140.4 | 141.65 | +0.89%
                Column ranges - without batch | 441.07 | 441.64 | +0.13%
                        Column ranges - batch | 112.67 | 109.67 | -2.66%

@sequba sequba closed this Oct 8, 2025
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.

1 participant