Bump actions/checkout from 3 to 4

.github/dependabot.yml

@@ -0,0 +1,19 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: weekly
+    rebase-strategy: auto
+
+  - package-ecosystem: "nuget" # See documentation for possible values
+    directory: "/" # Location of package manifests
+    schedule:
+      interval: "daily"
+    ignore:
+      - dependency-name: "NUnit"
+      - dependency-name: "NUnit3TestAdapter"
+      - dependency-name: "SonarAnalyzer.CSharp"
+      - dependency-name: "morelinq"
+      - dependency-name: "Microsoft.NET.Test.Sdk"
+

.github/workflows/dotnet.yml

@@ -0,0 +1,87 @@
+name: .NET build and test
+env:
+  CURRENT_VERSION: 0.96.${{ github.run_number }}
+  LAST_COMMIT_MESSAGE: ${{ github.event.head_commit.message }}
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  build:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup .NET
+      uses: actions/setup-dotnet@v3
+      with:
+        dotnet-version: 6.0.x
+    - name: Restore dependencies
+      run: dotnet restore
+    - name: Build
+      run: dotnet build --configuration Release --no-restore
+    - name: Test
+      run: dotnet test --no-build --verbosity normal --configuration Release
+    - name: Publish Unit Test Results
+      uses: actions/upload-artifact@v3
+      if: failure()
+      with:
+        name: TestResult
+        path: "TestResult/**/*"
+
+  deployRelease:
+    if: github.ref == 'refs/heads/release'
+    runs-on: windows-latest
+    needs: build
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup .NET
+      uses: actions/setup-dotnet@v3
+      with:
+        dotnet-version: 6.0.x
+    - name: Restore dependencies
+      run: dotnet restore
+    - name: Build
+      run: dotnet build --configuration Release --no-restore
+    - name: NugetPush
+      env: 
+          NUGET_TOKEN_EXISTS: ${{ secrets.NUGET_TOKEN }}
+      if:  env.NUGET_TOKEN_EXISTS != ''
+      run: |
+          ls ./ClosedXML/bin/Release
+          dotnet nuget push .\ClosedXML\bin\Release\*.nupkg --skip-duplicate --api-key ${{secrets.NUGET_TOKEN}} --source https://api.nuget.org/v3/index.json
+    - name: GithubReleasesPush
+      uses: "marvinpinto/action-automatic-releases@latest"
+      with:
+        repo_token: "${{ secrets.GITHUB_TOKEN }}"
+        automatic_release_tag: ${{ env.CURRENT_VERSION }}
+        prerelease: false
+        title: "Release Build ${{ env.CURRENT_VERSION }}"
+        files: |
+          ./ClosedXML/bin/Release/*.nupkg
+          ./ClosedXML/bin/Release/*.snupkg
+
+  deployTest:
+    if: github.ref == 'refs/heads/develop'
+    runs-on: windows-latest
+    needs: build
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup .NET
+      uses: actions/setup-dotnet@v3
+      with:
+        dotnet-version: 6.0.x
+    - name: Restore dependencies
+      run: dotnet restore
+    - name: Build
+      run: dotnet build --configuration Release --no-restore
+    - name: NugetPush
+      env: 
+          NUGET_TOKEN_EXISTS: ${{ secrets.NUGET_TEST_TOKEN }}
+      if:  env.NUGET_TOKEN_EXISTS != ''
+      run: |
+          ls ./ClosedXML/bin/Release
+          dotnet nuget push .\ClosedXML\bin\Release\*.nupkg --skip-duplicate --api-key ${{secrets.NUGET_TEST_TOKEN}} --source https://apiint.nugettest.org/v3/index.json

.vscode/extensions.json

@@ -0,0 +1,5 @@
+{
+    "recommendations": [
+        "ms-dotnettools.csharp"
+    ]
+}

.vscode/launch.json

@@ -0,0 +1,27 @@
+{
+   // Use IntelliSense to find out which attributes exist for C# debugging
+   // Use hover for the description of the existing attributes
+   // For further information visit https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md
+   "version": "0.2.0",
+   "configurations": [
+    {
+      "name": ".NET Core Launch (console)",
+      "type": "coreclr",
+      "request": "launch",
+      "preLaunchTask": "build",
+      // If you have changed target frameworks, make sure to update the program path.
+      "program": "${workspaceFolder}/OpenXmlToHtmlTests/bin/Debug/netcoreapp3.1/OpenXmlToHtmlTests.dll",
+      "args": [],
+      "cwd": "${workspaceFolder}/OpenXmlToHtmlTests",
+      // For more information about the 'console' field, see https://aka.ms/VSCode-CS-LaunchJson-Console
+      "console": "internalConsole",
+      "stopAtEntry": false
+    },
+        {
+            "name": ".NET Core Attach",
+            "type": "coreclr",
+            "request": "attach",
+            "processId": "${command:pickProcess}"
+        }
+    ]
+}

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+    "editor.formatOnSave": true
+}

.vscode/tasks.json

@@ -0,0 +1,15 @@
+{
+  "version": "2.0.0",
+  "tasks": [
+    {
+      "label": "update nuget",
+      "command": "${workspaceFolder}/.vscode/updateNuget.sh",
+      "args": [],
+      "group": "build",
+      "presentation": {
+        "reveal": "always"
+      },
+      "problemMatcher": "$msCompile"
+    }
+  ]
+}

.vscode/updateNuget.sh

@@ -0,0 +1,13 @@
+#!/bin/bash
+regex='PackageReference Include="([^"]*)" Version="([^"]*)"'
+find . -name "*.*proj" | while read proj; do
+    while read line; do
+        if [[ $line =~ $regex ]]; then
+            name="${BASH_REMATCH[1]}"
+            version="${BASH_REMATCH[2]}"
+            if [[ $version != *-* ]]; then
+                dotnet add "$proj" package "$name"
+            fi
+        fi
+    done <"$proj"
+done

CONTRIBUTING.md

@@ -1,20 +1,20 @@
 # Developer guidelines
+
 The [OpenXML specification](https://www.ecma-international.org/publications/standards/Ecma-376.htm) is a large and complicated beast. In order for ClosedXML, the wrapper around OpenXML, to support all the features, we rely on community contributions. Before opening an issue to request a new feature, we'd like to urge you to try to implement it yourself and log a pull request.
 
 Here are some tips.
 
-* Before starting a large pull request, log an issue and outline the problem and a broad outline of your solution. The maintainers will discuss the issue with you and possibly propose some alternative approaches to align with the ClosedXML development conventions. 
+* Before starting a large pull request, log an issue and outline the problem and a broad outline of your solution. The maintainers will discuss the issue with you and possibly propose some alternative approaches to align with the ClosedXML development conventions.
 * Please submit pull requests that are based on the `develop` branch.
 * Where possible, pull requests should include unit tests that cover as many uses cases as possible.
 * We recommend Visual Studio 2019 or higher as the development environment. If you do use Visual Studio, please install these extensions:
   * [CodeMaid](https://marketplace.visualstudio.com/items?itemName=SteveCadwallader.CodeMaid), for ensuring consistent code syntax, whitespace convention, etc.
   * If you use a version of Visual Studio lower than 2017, you should install [editorconfig](<https://marketplace.visualstudio.com/items?itemName=EditorConfigTeam.EditorConfig>) support. Read more about [EditorConfig](http://www.editorconfig.org).
 * We use 4 spaces for code indentation. This is the default in Visual Studio. Don't leave any trailing white space at the end of lines or files.
 
-
-
 ## Working with Excel file internals
-Excel files (`.xlsx` and `.xlsm`) are zip packages. You can easily verify this by renaming the extension any Excel file to `.zip` and opening the file in your favourite `.zip` file editor.
+
+Excel files (`.xlsx` and `.xlsm`) are zip packages. You can easily verify this by renaming the extension any Excel file to `.zip` and opening the file in your favorite `.zip` file editor.
 
 Internally, the file contains files (also known as parts) that represent different entities in the Excel framework, for example `workbook.xml` and `table1.xml`. The [OpenXML specification](https://www.ecma-international.org/publications/standards/Ecma-376.htm) documents all these parts and their contents.
 
@@ -24,24 +24,53 @@ Making changes to the ClosedXML code may change the input or output of the packa
 
 A ClosedXML developer will often want to compare the internals of 2 similar Excel files. For example if you want to compare the output of a specific package part before and after your code changes. The long, difficult way would be to extract the package parts of the 2 files and manually compare the relevant parts. To ease this, we recommend this tooling stack:
 
-- [Total Commander](https://www.ghisler.com/download.htm)
-- [WinMerge](http://winmerge.org/downloads) version `2.14.0`, because subsequent versions for [some reason](https://bitbucket.org/winmerge/winmerge/issues/152/displayxmlfiles-plugin-not-included-with) excludes the required `DisplayXMLFiles.dll` plugin.
-- Set Total Commander [to use WinMerge](https://superuser.com/questions/238039/can-i-replace-internal-diff-in-total-commander-with-a-custom-tool) as its compare tool.
-- In WinMerge, enable `Plugins > Automatic Prediffer`
+* [Total Commander](https://www.ghisler.com/download.htm)
+* [WinMerge](http://winmerge.org/downloads) version `2.14.0`, because subsequent versions for [some reason](https://bitbucket.org/winmerge/winmerge/issues/152/displayxmlfiles-plugin-not-included-with) excludes the required `DisplayXMLFiles.dll` plugin.
+* Set Total Commander [to use WinMerge](https://superuser.com/questions/238039/can-i-replace-internal-diff-in-total-commander-with-a-custom-tool) as its compare tool.
+* In WinMerge, enable `Plugins > Automatic Prediffer`
 
 Now, to compare 2 similar, but not exact Excel files:
 
-- In Total Commander, navigate to the 1st file in the left-hand pane and the 2nd file in the right-hand pane.
-- Press `Ctrl+PageDown` to "enter" the package. You should see, among others, a `[Content_Types].xml` file in both panes.
-- You can now compare all package parts by selecting `Commands > Synchronise Dirs...`. Press `Compare`. This will do a full, recursive comparison. You can filter out parts that are identical. 
-- You can select an item that differs and press `Ctrl+F3` to open the two parts in WinMerge and see the exact comparison of the part's contents. The XML files should automatically reformat/reindent to ease the comparison instead of showing the entire XML contents on a single line. This is the reason for requiring the `DisplayXMLFiles.dll` plugin.
-- In Total Commander, you can also navigate to specific files in the left-hand and right-hand panes and select `File > Compare by Content...`. This will open WinMerge directly.
-- Note that since WinMerge reformats the XML, it does so in a temporary file. If you make changes to the contents of any of the 2 panes in WinMerge and save the file, it will not be saved back into the Excel file.
+* In Total Commander, navigate to the 1st file in the left-hand pane and the 2nd file in the right-hand pane.
+* Press `Ctrl+PageDown` to "enter" the package. You should see, among others, a `[Content_Types].xml` file in both panes.
+* You can now compare all package parts by selecting `Commands > Synchronize Dirs...`. Press `Compare`. This will do a full, recursive comparison. You can filter out parts that are identical.
+* You can select an item that differs and press `Ctrl+F3` to open the two parts in WinMerge and see the exact comparison of the part's contents. The XML files should automatically reformat/reindent to ease the comparison instead of showing the entire XML contents on a single line. This is the reason for requiring the `DisplayXMLFiles.dll` plugin.
+* In Total Commander, you can also navigate to specific files in the left-hand and right-hand panes and select `File > Compare by Content...`. This will open WinMerge directly.
+* Note that since WinMerge reformats the XML, it does so in a temporary file. If you make changes to the contents of any of the 2 panes in WinMerge and save the file, it will not be saved back into the Excel file.
+
+#### Scripted diff preperation
+
+Powershell script to recursive extract and delete every xlsx in a directory.
+
+``` Powershell
+foreach ($file in Get-ChildItem -Recurse -Filter *.xlsx)
+  {
+  cp "$($file.FullName)" "$($file.FullName).zip"
+  Expand-Archive "$($file.FullName).zip" -DestinationPath "$($file.FullName).unzipped"
+  rm "$($file.FullName).zip"
+  rm "$($file.FullName)"
+  }
+```
+
+Commands to remove irrelevant XML format before comparing
+
+``` npm
+npm install -g prettier
+npm install -g prettier @prettier/plugin-xml
+prettier --write '**/*.xml'
+```
+
+On windows you can call winmerge using
+
+``` powershell
+& "C:\Program Files\WinMerge\WinMergeU.exe" .\Actual\ .\Expected\
+```
 
 ## Code conventions
+
 ClosedXML has a fairly large codebase and we therefore want to keep code revisions as clean and tidy as possible. It is therefore important not to introduce unnecessary whitespace changes in your commits.
 
 To ensure you follow the coding conventions, please do the following steps before you commit your code:
 
-- In Visual Studio, run `CodeMaid > Cleanup Active Document` or `Ctrl+M, Space` on each file that you have altered. This will ensure the correct whitespace consistency.
-- Some files, not all, have a header in the first line: `// Keep this file CodeMaid organised and cleaned`. For these files, also run `CodeMaid > Reorganize Active Document` or `Ctrl+M, Z`. This will reorder properties and methods alphabetically into a predetermined order. For example, public properties and methods will be organized before private properties and methods. Not all files require this yet. Please take note of the headers.
+* In Visual Studio, run `CodeMaid > Cleanup Active Document` or `Ctrl+M, Space` on each file that you have altered. This will ensure the correct whitespace consistency.
+* Some files, not all, have a header in the first line: `// Keep this file CodeMaid organized and cleaned`. For these files, also run `CodeMaid > Reorganize Active Document` or `Ctrl+M, Z`. This will reorder properties and methods alphabetically into a predetermined order. For example, public properties and methods will be organized before private properties and methods. Not all files require this yet. Please take note of the headers.

ClosedXML.Examples/.editorconfig

@@ -0,0 +1,5 @@
+# C# files
+[*.cs]
+
+# S1481: Unused local variables should be removed
+dotnet_diagnostic.S1481.severity = none