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

GitAuto: [FEATURE] Refactor to use HttpClientFactory for HTTP client management #355

Closed
wants to merge 17 commits into from

Merge branch 'main' into gitauto/issue-231-df25476e-1d99-44af-94b4-5e…

fb855f0
Select commit
Loading
Failed to load commit list.
Closed

GitAuto: [FEATURE] Refactor to use HttpClientFactory for HTTP client management #355

Merge branch 'main' into gitauto/issue-231-df25476e-1d99-44af-94b4-5e…
fb855f0
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Nov 12, 2024 in 0s

4 new issues (0 max.) of at least minor severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 4
           

Complexity increasing per file
==============================
- Src/VTEX/Startup.cs  1
         

Complexity decreasing per file
==============================
+ Src/VTEX.Health/VtexHealthClient.cs  -2

See the complete overview on Codacy

Annotations

Check warning on line 37 in Src/VTEX.Health/VtexHealthClient.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

Src/VTEX.Health/VtexHealthClient.cs#L37 

This class can't be instantiated; make its constructor 'public'.

Check warning on line 78 in Src/VTEX.Health/VtexHealthClient.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

Src/VTEX.Health/VtexHealthClient.cs#L78 

Remove this useless assignment to local variable 'responseContent'.

Check warning on line 84 in Src/VTEX.Health/VtexHealthClient.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

Src/VTEX.Health/VtexHealthClient.cs#L84 

Remove this useless assignment to local variable 'response'.

Check notice on line 60 in Src/VTEX/VTEXWrapper.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

Src/VTEX/VTEXWrapper.cs#L60 

Remove unassigned field '_accountName', or set its value.
View more details on Codacy Production