#200 implement dotnet best practises

[thompson-tomo]

Description

• Move to central package management
• Use central properties to prepare for split
• Move to csproj internalvisibleto rather than assemblyinfo.cs

Reason for the change
Lays the foundation for future changes in particular package splitting.

Related issue(s)
Closes #200

[yurvon-screamo]

I will definitely accept this request when we resolve the issues with the code style and architecture)

[yurvon-screamo]

Hi, i ready to merge this. @thompson-tomo will you resolve conflict?

[thompson-tomo]

Sure will do so in the next couple of days

[thompson-tomo]

@yurvon-screamo sorry for the delay, find merges resolved. Note i am guessing the large diff is due to the editor config now being run on the csproj files etc

[yurvon-screamo]

in .github/npm/action.yaml try change, maybe it will help:

    - name: Run NPM install
      shell: sh
      run: npm ci
      working-directory: ./src/Saunter.UI

to:

    - name: Create package-lock
      shell: sh
      run: npm install --package-lock-only
      working-directory: ./src/Saunter.UI
    - name: Run NPM install
      shell: sh
      run: npm ci
      working-directory: ./src/Saunter.UI