Skip to content
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

Update Project Build Configuration #50

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

Conversation

myty
Copy link
Contributor

@myty myty commented Mar 17, 2023

Addresses: #59 and #62

This PR first started out as just upgrading to PNPM from NPM, but has since morphed into a configuration update for the entire project.

The plan is to give the repo some loving care, and this is the first stop.

Next stop is to add C# formatting like we do with prettier for frontend code, then it will be to address the Do...Try class and pattern.

  • Related GitHub issue(s) linked in PR description
  • Destination branch merged, built and tested with your changes
  • Code formatted and follows best practices and patterns
  • Code builds cleanly (no additional warnings or errors)
  • Manually tested
  • Automated tests are passing
  • No decreases in automated test coverage
  • Documentation updated (readme, docs, comments, etc.)
  • Localization: No hard-coded error messages in code files (minimally in string constants)

@myty myty requested a review from klauffer March 17, 2023 12:12
@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #50 (d777ee1) into main (7d9d4da) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
- Coverage   59.55%   59.52%   -0.04%     
==========================================
  Files         106      106              
  Lines        3071     3071              
  Branches      330      330              
==========================================
- Hits         1829     1828       -1     
- Misses       1210     1211       +1     
  Partials       32       32              

see 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@myty myty self-assigned this Oct 14, 2023
myty added 10 commits May 13, 2024 12:40
- Update .prettierignore to include *.pnpm-lock.yaml
- Update .prettierrc to change tabWidth from 4 to 2
- Update .editorconfig to change indent_size from 4 to 2 for all files except *.cs
- Update .config/dotnet-tools.json to use the latest versions of husky and dotnet-outdated-tool
- Update .github/workflows/build.yaml to fix indentation and add comments
- Update Microsoft.AspNetCore.Http to version 2.2.2 in Extensions.Tests.csproj
- Update Microsoft.SourceLink.GitHub to version 8.0.0 in Directoy.Build.props
- Update Microsoft.Extensions.Configuration.Abstractions to version 8.0.0 in Extensions.csproj
- Update Microsoft.Extensions.Configuration.Binder to version 8.0.1 in Extensions.csproj
- Update Microsoft.CSharp to version 4.7.0 in Extensions.csproj
- Update Microsoft.Net.Http.Headers to version 2.2.8 in Extensions.csproj
- Update Newtonsoft.Json to version 13.0.3 in Extensions.csproj
- Update AutoMapper to version 10.1.1 in Testing.csproj
- Update Bogus to version 35.5.1 in Testing.csproj
- Update Microsoft.Extensions.Logging to version 8.0.0 in Testing.csproj
- Update Moq to version 4.20.70 in Testing.csproj
- Update Shouldly to version 4.2.1 in Testing.csproj
- Update xunit to version 2.8.0 in Testing.csproj
- Update MailKit to version 4.5.0 in Core.csproj
- Update Microsoft.AspNetCore.Cryptography.KeyDerivation to version 8.0.4 in Core.csproj
- Update Microsoft.Extensions.Caching.Abstractions to version 8.0.0 in Core.csproj
- Update Microsoft.Extensions.Logging to version 8.0.0 in Core.csproj
- Update Microsoft.Extensions.Localization.Abstractions to version 8.0.4 in Core.csproj
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.

1 participant