Skip to content

Analyze

Analyze #44

Triggered via schedule September 20, 2024 00:07
Status Success
Total duration 4m 10s
Artifacts

analyze.yml

on: schedule
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Analyze
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v2, github/codeql-action/init@v2, github/codeql-action/analyze@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Analyze
CodeQL Action v2 will be deprecated on December 5th, 2024. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2024-01-12-code-scanning-deprecation-of-codeql-action-v2/
Analyze: LostFilmMonitoring.DAO.Interfaces/DomainModels/FeedItem.cs#L29
When implementing IComparable<T>, you should also override ==, !=, <, <=, >, and >=. (https://rules.sonarsource.com/csharp/RSPEC-1210)
Analyze: LostFilmMonitoring.DAO.Interfaces/DomainModels/Series.cs#L47
Constructor has 13 parameters, which is greater than the 7 authorized. (https://rules.sonarsource.com/csharp/RSPEC-107)
Analyze: LostFilmMonitoring.DAO.Interfaces/DomainModels/FeedItem.cs#L94
Remove this unnecessary check for null. (https://rules.sonarsource.com/csharp/RSPEC-2589)
Analyze: LostFilmTV.Client/Exceptions/RemoteServiceUnavailableException.cs#L33
Update this implementation of 'ISerializable' to conform to the recommended serialization pattern. Add a 'protected' constructor 'RemoteServiceUnavailableException(SerializationInfo, StreamingContext)'. (https://rules.sonarsource.com/csharp/RSPEC-3925)
Analyze: LostFilmTV.Client/Exceptions/RemoteServiceUnavailableException.cs#L33
'Exception' implements 'ISerializable' so 'ISerializable' can be removed from the inheritance list. (https://rules.sonarsource.com/csharp/RSPEC-1939)
Analyze: LostFilmTV.Client/Response/FeedItemResponse.cs#L29
When implementing IComparable<T>, you should also override ==, !=, <, <=, >, and >=. (https://rules.sonarsource.com/csharp/RSPEC-1210)
Analyze: LostFilmTV.Client/RssFeed/LostFilmRssFeed.cs#L31
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
Analyze: LostFilmTV.Client/RssFeed/ReteOrgRssFeed.cs#L31
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
Analyze: LostFilmTV.Client/RssFeed/BaseRssFeed.cs#L128
Pass a timeout to limit the execution time. (https://rules.sonarsource.com/csharp/RSPEC-6444)
Analyze: LostFilmTV.Client/Response/FeedItemResponse.cs#L69
Pass a timeout to limit the execution time. (https://rules.sonarsource.com/csharp/RSPEC-6444)