Skip to content

Kentico tag manager

Kentico tag manager #3

Triggered via pull request January 12, 2024 09:28
Status Success
Total duration 1m 13s
Billable time 2m
Artifacts

ci.yml

on: pull_request
Build and Test
1m 2s
Build and Test
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Build and Test: src/Kentico.Xperience.TagManager/Admin/UIPages/CodeSnippetModelCreate.cs#L90
Change return type to 'void'; not a single caller uses the returned value. (https://rules.sonarsource.com/csharp/RSPEC-3241)
Build and Test: examples/DancingGoat/Models/Reusable/Cafe/CafeViewModel.cs#L8
XML comment has badly formed XML -- 'A string literal was expected, but no opening quotation mark was found.'
Build and Test: examples/DancingGoat/Models/WebPage/ArticlesSection/ArticlesSectionViewModel.cs#L8
XML comment has badly formed XML -- 'A string literal was expected, but no opening quotation mark was found.'
Build and Test: examples/DancingGoat/Controllers/DancingGoatPrivacyController.cs#L127
'ConsentInfoExtensions.GetConsentText(ConsentInfo, string)' is obsolete: 'Use Task-based method override instead.'
Build and Test: examples/DancingGoat/Models/ContentRepositoryBase.cs#L30
XML comment has a param tag for 'pageRetriever', but there is no parameter by that name
Build and Test: examples/DancingGoat/DataProtectionSamples/PersonalDataCollectors/SampleContactDataCollectorCore.cs#L121
Private classes which are not derived in the current assembly should be marked as 'sealed'. (https://rules.sonarsource.com/csharp/RSPEC-3260)
Build and Test: examples/DancingGoat/DataProtectionSamples/PersonalDataCollectors/SampleContactDataCollectorCore.cs#L420
All 'WriteConsents' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)
Build and Test: examples/DancingGoat/Resources/SharedResources.cs#L3
Remove this empty class, write its code or make it an "interface". (https://rules.sonarsource.com/csharp/RSPEC-2094)
Build and Test: examples/DancingGoat/Components/Widgets/CTAButton/CTAButtonWidgetProperties.cs#L11
Rename class 'CTAButtonWidgetProperties' to match pascal case naming rules, consider using 'CtaButtonWidgetProperties'. (https://rules.sonarsource.com/csharp/RSPEC-101)