Analyze #51
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/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/LostFilmRssFeed.cs#L31
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
Analyze:
LostFilmTV.Client/Response/FeedItemResponse.cs#L69
Pass a timeout to limit the execution time. (https://rules.sonarsource.com/csharp/RSPEC-6444)
|
Analyze:
LostFilmTV.Client/Response/FeedItemResponse.cs#L72
Pass a timeout to limit the execution time. (https://rules.sonarsource.com/csharp/RSPEC-6444)
|