Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update ClosedParser to 2.0 (bang reference) #2596

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

Merged
merged 1 commit into from
Dec 27, 2024
Merged

Update ClosedParser to 2.0 (bang reference) #2596

merged 1 commit into from
Dec 27, 2024

Conversation

jahav
Copy link
Member

@jahav jahav commented Dec 26, 2024

Parser version 2.0 has two new methods in the IAstFactory interface for bang references (used only in defined names). Adding methods to interface is a breaking change, so major version was increased to 2.

Bang reference (e.g. !$B$7:$D$8) are

  • now parseable and formulas with it won't throw during sheet rename and delete or formula conversion from A1 to R1C1 and back.
  • calc engine doesn't evaluate them, they are represented as NotSupportedNode for now.

There is also a method for bang name (!Data), but it's not yet parseable, but will be.

ClosedXML.Parser 2.0.0-preview1 package is unlisted on nuget.org.

The 2.0 version has two new methods in IAstFactory. BangReference and
BangName. BangName is not actually parsed yet, but will be.
@jahav jahav added this to the v0.105.0 milestone Dec 26, 2024
@jahav jahav merged commit b6394d5 into develop Dec 27, 2024
6 checks passed
@jahav jahav deleted the update-parser branch December 27, 2024 00:03
@rajtilakvyas
Copy link

@jahav I have been using ClosedXML 0.105.0-rc, which i downloaded from https://ci.appveyor.com/project/ClosedXML/ClosedXML/branch/develop/artifacts

which needs ClosedXML.Parser2.0.0-preview1, from where i can get that, i could not find on nuget.

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.

2 participants