Skip to content

Split expression configuration from runtime parameters - #600

Merged
gumbarros merged 28 commits into
masterfrom
expression-configuration
Jul 14, 2026
Merged

gumbarros merged 28 commits into
masterfrom
expression-configuration

Conversation

@gumbarros

@gumbarros gumbarros commented Jul 10, 2026

Copy link
Copy Markdown
Member

This closes #597

I also decided to keep the ExpressionOptions enum. Rewriting all the tests would be a massive amount of work, and I also think removing it would be a pretty nuclear breaking change for the community.

Ex:

The average code that is:

var expression = new Expression("2+2", ExpressionOptions.DecimalAsDefault);

Will work fine, but there is lots of breaking changes.

@gumbarros
gumbarros requested a review from Bykiev July 10, 2026 19:55
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Dismissed
gumbarros and others added 3 commits July 10, 2026 17:04
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Comment thread src/NCalc.Core/Expression.cs Fixed
Comment thread src/NCalc.Core/Expression.cs Fixed
@gumbarros

Copy link
Copy Markdown
Member Author

@github-code-quality, any notes clanker?

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.

Refactor ExpressionOptions flags into multiple smaller classes

1 participant