Big refactoring before first beta release

[michelebastione]

Given our decision to move forwards with a v2 release, I tought this was a good moment to implement a number of changes to the codebase, some long overdue:

• Bumped version to 2.0.0-beta.1
• Bumped language version to 13 to make use of new syntax features
• Added suuport for nullable reference types
• Changed block scoped namespaces to file scoped
• Changed namespace MiniExcelLibs.OpenXml.SaveByTemplate to MiniExcelLibs.SaveByTemplate
• Changed using statements to using declarations where possible and appropriate
• Implemented collections expressions where possible and appropriate
• Changed == null and != null checks to is null and is not null
• Implemented pattern matching and primary constructors where possible and appropriate
• Removed License code file
• Added the TaskHelper class
• Renamed OpenXml.Config class to OpenXml.Constants.Schemas
• Renamed IConfiguration and Configuration classes to IMiniExcelConfiguration and MiniExcelConfiguration
• Renamed calChainHelper.cs file to CalcChainHelper.cs
• Fixes issues 814, 815 and 817 with commit 1103aedda by @shps951023
• Resolves conflicting TargetFrameworks in main project file
• Upgrades ClosedXml and Epplus packages in test project
• Adds EpplusLicense class that specifies the context as non commercial
• Adds codeql analysis to pushes and PRs of maintenance branch
• Fixes typo in drawio diagram
• Minor style changes (sorting modifiers, renaming variables, etc...)
• Removed MiniExcelTask.cs, it was only included for compatibility with .net45

I would also like to separate everything in 3 main assemblies: MiniExcelLibs.Core, MiniExcelLibs.OpenXml and MiniExcelLibs.Csv. I'm not sure yet if it's approprate to add this other big change here or to make a subsequent PR.
Let me know what you guys think about it all.

[michelebastione]

On further thought, separating the assemblies will be a big enough endeavour on its own, so it's probably for the best if we take our time with it.

[izanhzh]

Reply

A very good and very big refactoring. I like it ❤. The following are some of my simple opinions:

• Added suport for nullable reference types

  Great, and we still need to make extensive code adjustments to eliminate all compilation warning messages (e.g. CS8601、CS8604 ... ). Let's handle these improvements in follow-up PRs.
  

• Changed block scoped namespaces to file scoped

  I added a configuration item to .editorconfig to make VS use file-scoped style by default when creating new files

• I would also like to separate everything in 3 main assemblies: MiniExcelLibs.Core, MiniExcelLibs.OpenXml and MiniExcelLibs.Csv

  Super nice. It is recommended to set a unified RootNamespace MiniExcelLibs for all 3 main assemblies through the Directory.Build.props file. This will help maintain consistency across the projects and simplify future maintenance.

Others

I've added one more change: Removed MiniExcelTask.cs