Skip to content

Text splitter improvements #100

Text splitter improvements

Text splitter improvements #100

Triggered via pull request January 26, 2024 12:28
Status Success
Total duration 2m 42s
Artifacts 2

main.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

7 warnings
CI
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-dotnet@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
CI: src/Encamina.Enmarcha.AI.Abstractions/ITextSplitter.cs#L28
Split this 201 characters long line (which is greater than 200 authorized). (https://rules.sonarsource.com/csharp/RSPEC-103)
CI: src/Encamina.Enmarcha.AI.Abstractions/TextSplitter.cs#L96
Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
CI: src/Encamina.Enmarcha.AI/TextSplitters/RecursiveCharacterTextSplitter.cs#L34
Refactor the code in order to remove this break statement. (https://rules.sonarsource.com/csharp/RSPEC-1227)
CI: src/Encamina.Enmarcha.AI.Abstractions/ITextSplitter.cs#L28
Split this 201 characters long line (which is greater than 200 authorized). (https://rules.sonarsource.com/csharp/RSPEC-103)
CI: src/Encamina.Enmarcha.AI.Abstractions/TextSplitter.cs#L96
Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
CI: src/Encamina.Enmarcha.AI/TextSplitters/RecursiveCharacterTextSplitter.cs#L34
Refactor the code in order to remove this break statement. (https://rules.sonarsource.com/csharp/RSPEC-1227)

Artifacts

Produced during runtime
Name Size
code-coverage-report Expired
384 KB
enmarcha-libraries-100 Expired
1.06 MB