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.

Sign up for GitHub

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

Jump to bottom

Solution improvements #623

Merged
merged 3 commits into from
Mar 29, 2024
Merged

Solution improvements #623

merged 3 commits into from
Mar 29, 2024

Conversation

martincostello
Copy link
Owner

  • Enable (almost) all built-in style analyzers.
  • Fix new style warnings.
  • Use global package references.
  • Add reference trimmer analyzer.

@martincostello
Solution improvements
86ae65a 
- Enable (almost) all built-in style analyzers.
- Fix new style warnings.
- Use global package references.
- Add reference trimmer analyzer.
@martincostello martincostello added enhancement New feature or request dependencies Pull requests that update a dependency file .NET Pull requests that update .net code labels Mar 29, 2024
@martincostello martincostello enabled auto-merge (rebase) March 29, 2024 11:26
@martincostello
Fix formatting warnings
5edcd66 
Avoid formatting warnings that only seem to happen in CI.
@martincostello
Fix references
de95a82 
Fix missing package references.
@codecov Codecov
Copy link

codecov bot commented Mar 29, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.25%. Comparing base (e1dddf5) to head (de95a82).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #623   +/-   ##
=======================================
  Coverage   98.25%   98.25%           
=======================================
  Files           8        8           
  Lines         287      287           
  Branches       30       30           
=======================================
  Hits          282      282           
  Misses          1        1           
  Partials        4        4
Flag Coverage Δ
linux 97.56% <100.00%> (ø)
macos 97.56% <100.00%> (ø)
windows 98.25% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martincostello martincostello merged commit 9b7780f into main Mar 29, 2024
12 checks passed
@martincostello martincostello deleted the solution-improvements branch March 29, 2024 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request .NET Pull requests that update .net code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@martincostello