Fixes & Improvements: logs, dedupe, private methods #12
ElanisAnnotations
10 warnings
|
build:
Lumen.Modules.Youtube.Common/YoutubeAPIHelper.cs#L8
Rename class 'YoutubeAPIHelper' to match pascal case naming rules, consider using 'YoutubeApiHelper'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
Lumen.Modules.Youtube.Common/YoutubeAPIHelper.cs#L42
Assign this magic number '50' to a well-named variable or constant, and use that instead. (https://rules.sonarsource.com/csharp/RSPEC-109)
|
|
build:
Lumen.Modules.Youtube.Common/YoutubeAPIHelper.cs#L9
Add the 'Async' suffix to the name of this method. (https://rules.sonarsource.com/csharp/RSPEC-4261)
|
|
build:
Lumen.Modules.Youtube.Tests/YoutubeAPIHelperTest.cs#L15
Possible null reference assignment.
|
|
build:
Lumen.Modules.Youtube.Tests/YoutubeAPIHelperTest.cs#L16
Possible null reference assignment.
|
|
build:
Lumen.Modules.Youtube.Tests/YoutubeAPIHelperTest.cs#L10
Non-nullable field 'API_KEY' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
build:
Lumen.Modules.Youtube.Tests/YoutubeAPIHelperTest.cs#L10
Non-nullable field 'PLAYLIST_ID' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
build:
Lumen.Modules.Youtube.Data/Migrations/20250214144308_InitialSchema.cs#L22
Remove this default value assigned to parameter 'nullable'. (https://rules.sonarsource.com/csharp/RSPEC-3254)
|
|
build:
Lumen.Modules.Youtube.Data/Migrations/20250214144308_InitialSchema.cs#L23
Remove this default value assigned to parameter 'nullable'. (https://rules.sonarsource.com/csharp/RSPEC-3254)
|
|
build:
Lumen.Modules.Youtube.Data/Migrations/20250214144308_InitialSchema.cs#L24
Remove this default value assigned to parameter 'nullable'. (https://rules.sonarsource.com/csharp/RSPEC-3254)
|