refactor: reduce METAR chunk decoder complexity#82
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
Qodana Community for .NET5 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked View the detailed Qodana reportTo be able to view the detailed Qodana report, you can either:
To get - name: 'Qodana Scan'
uses: JetBrains/[email protected]
with:
upload-result: trueContact Qodana teamContact us at [email protected]
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
1 participant
All Submissions:
Changes to Core Features:
CloudChunkDecoder,VisibilityChunkDecoder,RunwayVisualRangeChunkDecoder, andWindShearChunkDecoderby extracting parsing/validation helpers and keepingParse()focused on orchestration.dotnet build MetarDecoder.sln --configuration Releasedotnet test tests/Metar.Decoder.Tests/Metar.Decoder.Tests.csproj --configuration Release --framework net8.0 --collect:"XPlat Code Coverage"dotnet test tests/Metar.Decoder.Tests/Metar.Decoder.Tests.csproj --configuration Release --framework net10.0 --collect:"XPlat Code Coverage"dotnet test tests/Taf.Decoder.Tests/Taf.Decoder.Tests.csproj --configuration Release --framework net8.0 --collect:"XPlat Code Coverage"dotnet test tests/Taf.Decoder.Tests/Taf.Decoder.Tests.csproj --configuration Release --framework net10.0 --collect:"XPlat Code Coverage"Link to Devin session: https://app.devin.ai/sessions/a4e91bc6cd1447788a7bec8c00406d09