KeYProject / key

KeY Theorem Prover for Deductive Java Verification
https://key-project.org
Other
41 stars 24 forks source link

fixes #1533 #3423

Closed JonasKlamroth closed 5 months ago

JonasKlamroth commented 5 months ago

Related Issue

This pull request addresses #1533 .

Intended Change

This replaces the fix seperator "," with a static final String which may be set arbitrarily. This resolved the described problems in the description of the issue.

Type of pull request

Ensuring quality

Additional information and contact(s)

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 37.91%. Comparing base (1fb0c10) to head (dd1a99f). Report is 11 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3423 +/- ## ============================================ + Coverage 37.77% 37.91% +0.14% - Complexity 17031 17069 +38 ============================================ Files 2076 2076 Lines 126950 127019 +69 Branches 21381 21388 +7 ============================================ + Hits 47952 48164 +212 + Misses 73092 72949 -143 Partials 5906 5906 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

wadoon commented 5 months ago

force pushed for reformatting

wadoon commented 5 months ago

On the other side, we might want to migrate from csv to something more robust like Json. The ci-tool offers already such an export for further script-based analysis.