Skip to content

Conversation

@LucaFue
Copy link
Collaborator

@LucaFue LucaFue commented Dec 3, 2025

To ensure consistency and quality in the existing set of all statements, a quality control is done, focusing on:

  • Correct spelling of the statements, especially of those already existing/those that were created a longer time ago
  • Unify the indentation levels of evidences and references of all statements
  • Improve the grammatical formulation and readability of existing statements

Changes like behavior -> behaviour or analyzed -> analyzed have been made for consistency between US and UK english. We will stick to UK english, since TSF is developed by a british company and therefore UK english has already been used in e.g. the formulation of the TAs and TTs.

@LucaFue LucaFue marked this pull request as draft December 3, 2025 09:56
@github-actions github-actions bot added documentation Improvements or additions to documentation L tests CMake labels Dec 3, 2025

TEST_CASE("iterators 1")
{
SECTION("basic behavior")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is somewhat dangerous. Please refrain from search and replace in files that are directly from nlohmann/json.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@coveralls
Copy link

coveralls commented Dec 3, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling ebfd1de on LucaFgr-Perform_quality_control_on_all_statements
into df946d2 on main.

@github-actions github-actions bot removed the CMake label Dec 3, 2025
@github-actions github-actions bot removed the L label Dec 4, 2025
@github-actions github-actions bot added the L label Dec 4, 2025
@github-actions github-actions bot removed the L label Dec 5, 2025
…ments

Signed-off-by: LucaFue <luca.fueger@d-fine.de>
@github-actions github-actions bot added the L label Dec 9, 2025
@LucaFue LucaFue marked this pull request as ready for review December 9, 2025 07:19
@github-actions github-actions bot removed the L label Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants