patch-1 - Branch - cpp-linter/cpp-linter-hooks

fix: get rid of `toml` and `packaging` and improve performance

#89Merged

Comparing

patch-1

(

0a4c0ba

) with

main

(

90a453d

)

-60%

Improvements: 39

Regressions: 5

Untouched: 18

New: 6

Dropped: 6

Ignored: 0

Benchmarks

Failed

test_resolve_version_clang_tidy[19-19.1.0.1]Missing

tests/test_util.py::test_resolve_version_clang_tidy[19-19.1.0.1]

N/A

450.8 µs

N/A

test_resolve_version_clang_tidy[18-18.1.8]Missing

tests/test_util.py::test_resolve_version_clang_tidy[18-18.1.8]

N/A

446.5 µs

N/A

test_resolve_version_clang_format[20.1-20.1.7]Missing

tests/test_util.py::test_resolve_version_clang_format[20.1-20.1.7]

N/A

2.1 ms

N/A

test_resolve_version_clang_format[20-20.1.7]Missing

tests/test_util.py::test_resolve_version_clang_format[20-20.1.7]

N/A

1.8 ms

N/A

test_resolve_version_clang_format[18.1-18.1.8]Missing

tests/test_util.py::test_resolve_version_clang_format[18.1-18.1.8]

N/A

2.1 ms

N/A

test_resolve_version_clang_format[18-18.1.8]Missing

tests/test_util.py::test_resolve_version_clang_format[18-18.1.8]

N/A

1.8 ms

N/A

test_get_version_from_dependency_malformed_tomlRegression

tests/test_util.py::test_get_version_from_dependency_malformed_toml

-10%

1.3 ms

1.4 ms

test_get_version_from_dependency_missing_dependencyRegression

tests/test_util.py::test_get_version_from_dependency_missing_dependency

-10%

1.3 ms

1.4 ms

test_get_version_from_dependency_successRegression

tests/test_util.py::test_get_version_from_dependency_success

-11%

1.4 ms

1.6 ms

test_resolve_install_invalid_versionRegression

tests/test_util.py::test_resolve_install_invalid_version

-13%

936 µs

1,074.6 µs

test_resolve_version_clang_format[invalid-None]Regression

tests/test_util.py::test_resolve_version_clang_format[invalid-None]

-60%

103.2 µs

256.6 µs

Improved

test_resolve_version_clang_format[20.1.7-20.1.7]

tests/test_util.py::test_resolve_version_clang_format[20.1.7-20.1.7]

×18

1,669.2 µs

93.5 µs

test_resolve_version_clang_format[20.99-None]

tests/test_util.py::test_resolve_version_clang_format[20.99-None]

×8.1

2,077.7 µs

257.8 µs

test_resolve_version_clang_format[99-None]

tests/test_util.py::test_resolve_version_clang_format[99-None]

×6.9

1,800.6 µs

259.5 µs

test_resolve_version_clang_tidy[20-20.1.0]

tests/test_util.py::test_resolve_version_clang_tidy[20-20.1.0]

×4.7

445.2 µs

95.1 µs

test_run_clang_format_invalid[args4-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args4-1]

×3

2,556.9 µs

853.5 µs

test_run_clang_format_invalid[args5-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args5-1]

×3

2,542.7 µs

859.5 µs

test_run_clang_format_dry_run[args0-1]

tests/test_clang_format.py::test_run_clang_format_dry_run[args0-1]

×2.9

2,382.5 µs

824.5 µs

test_run_clang_format_invalid[args0-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args0-1]

×2.8

2,391.7 µs

839.7 µs

test_ensure_installed_no_runtime_version

tests/test_util.py::test_ensure_installed_no_runtime_version

×2.8

2,431.3 µs

855.7 µs

test_run_clang_format_invalid[args2-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args2-1]

×2.8

2,646.5 µs

941.4 µs

test_run_clang_format_invalid[args1-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args1-1]

×2.8

2,630 µs

942 µs

test_run_clang_format_invalid[args3-1]

tests/test_clang_format.py::test_run_clang_format_invalid[args3-1]

×2.7

2,563.6 µs

944 µs

test_resolve_install_tool_already_installed_correct_version

tests/test_util.py::test_resolve_install_tool_already_installed_correct_version

×2.7

2,476.8 µs

913 µs

test_run_clang_format_valid[args4-expected_retval4]

tests/test_clang_format.py::test_run_clang_format_valid[args4-expected_retval4]

×2.7

2,678.3 µs

992.5 µs

test_run_clang_format_valid[args5-expected_retval5]

tests/test_clang_format.py::test_run_clang_format_valid[args5-expected_retval5]

×2.7

2,672.9 µs

995.5 µs

test_resolve_install_tool_not_installed

tests/test_util.py::test_resolve_install_tool_not_installed

×2.7

2,508.8 µs

935.5 µs

test_run_clang_format_verbose_error

tests/test_clang_format.py::test_run_clang_format_verbose_error

×2.6

2,478.5 µs

935.5 µs

test_run_clang_format_valid[args1-expected_retval1]

tests/test_clang_format.py::test_run_clang_format_valid[args1-expected_retval1]

×2.6

2.8 ms

1.1 ms

test_run_clang_format_valid[args2-expected_retval2]

tests/test_clang_format.py::test_run_clang_format_valid[args2-expected_retval2]

×2.6

2.8 ms

1.1 ms

test_run_clang_format_valid[args0-expected_retval0]

tests/test_clang_format.py::test_run_clang_format_valid[args0-expected_retval0]

×2.6

2.6 ms

1 ms

test_run_clang_format_verbose

tests/test_clang_format.py::test_run_clang_format_verbose

×2.5

2.6 ms

1 ms

test_run_clang_format_valid[args3-expected_retval3]

tests/test_clang_format.py::test_run_clang_format_valid[args3-expected_retval3]

×2.5

2.7 ms

1.1 ms

test_ensure_installed[clang-format-20]

tests/test_util.py::test_ensure_installed[clang-format-20]

×2.1

2.9 ms

1.3 ms

test_ensure_installed_version_mismatch

tests/test_util.py::test_ensure_installed_version_mismatch

×2

3.1 ms

1.5 ms

test_resolve_version_clang_tidy[99-None]

tests/test_util.py::test_resolve_version_clang_tidy[99-None]

+78%

431.9 µs

242.6 µs

test_resolve_install_tool_version_mismatch

tests/test_util.py::test_resolve_install_tool_version_mismatch

+74%

3.6 ms

2.1 ms

test_run_clang_tidy_invalid[args5-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args5-1]

+45%

1,082.9 µs

745.5 µs

test_run_clang_tidy_invalid[args1-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args1-1]

+45%

1,089.8 µs

750.2 µs

test_run_clang_tidy_invalid[args4-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args4-1]

+45%

1,085.4 µs

749.5 µs

test_run_clang_tidy_invalid[args2-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args2-1]

+45%

1,084.3 µs

749.6 µs

test_run_clang_tidy_invalid[args0-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args0-1]

+44%

1,044.4 µs

726.7 µs

test_run_clang_tidy_invalid[args3-1]

tests/test_clang_tidy.py::test_run_clang_tidy_invalid[args3-1]

+44%

1,084 µs

754.5 µs

test_run_clang_tidy_valid[args4-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args4-1]

+42%

1,147.3 µs

806.8 µs

test_run_clang_tidy_valid[args2-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args2-1]

+42%

1,140.6 µs

803.5 µs

test_run_clang_tidy_valid[args5-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args5-1]

+42%

1,140.2 µs

805 µs

test_run_clang_tidy_valid[args3-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args3-1]

+42%

1,138.7 µs

804.1 µs

test_run_clang_tidy_valid[args1-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args1-1]

+41%

1,146.6 µs

811.6 µs

test_run_clang_tidy_valid[args0-1]

tests/test_clang_tidy.py::test_run_clang_tidy_valid[args0-1]

+40%

1,107.7 µs

790.1 µs

test_ensure_installed[clang-tidy-20]

tests/test_util.py::test_ensure_installed[clang-tidy-20]

+28%

1.5 ms

1.2 ms

New

test_resolve_version_clang_format[18-18.1.0]

tests/test_util.py::test_resolve_version_clang_format[18-18.1.0]

N/A

105.5 µs

test_resolve_version_clang_format[18.1-18.1.0]

tests/test_util.py::test_resolve_version_clang_format[18.1-18.1.0]

N/A

105.8 µs

test_resolve_version_clang_format[20-20.1.0]

tests/test_util.py::test_resolve_version_clang_format[20-20.1.0]

N/A

109 µs

test_resolve_version_clang_format[20.1-20.1.0]

tests/test_util.py::test_resolve_version_clang_format[20.1-20.1.0]

N/A

108.8 µs

test_resolve_version_clang_tidy[18-18.1.1]

tests/test_util.py::test_resolve_version_clang_tidy[18-18.1.1]

N/A

93.8 µs

test_resolve_version_clang_tidy[19-19.1.0]

tests/test_util.py::test_resolve_version_clang_tidy[19-19.1.0]

N/A

94.4 µs

Passed

test_ensure_installed[clang-tidy-None]

tests/test_util.py::test_ensure_installed[clang-tidy-None]

+1%

1.2 ms

test_is_installed_not_found

tests/test_util.py::test_is_installed_not_found

538.5 µs

536.3 µs

test_get_runtime_version_clang_format

tests/test_util.py::test_get_runtime_version_clang_format

512.2 µs

510.8 µs

test_get_runtime_version_exception

tests/test_util.py::test_get_runtime_version_exception

519.7 µs

518.8 µs

test_ensure_installed[clang-format-None]

tests/test_util.py::test_ensure_installed[clang-format-None]

1.2 ms

test_ensure_installed_tool_not_found

tests/test_util.py::test_ensure_installed_tool_not_found

1 ms

test_resolve_install_no_version_specified

tests/test_util.py::test_resolve_install_no_version_specified

932.1 µs

930.8 µs

test_install_tool_success_but_not_found

tests/test_util.py::test_install_tool_success_but_not_found

1.1 ms

test_get_runtime_version_clang_tidy_single_line

tests/test_util.py::test_get_runtime_version_clang_tidy_single_line

509.7 µs

509.4 µs

test_is_installed_with_shutil_which

tests/test_util.py::test_is_installed_with_shutil_which

603.4 µs

603 µs

test_install_tool_success

tests/test_util.py::test_install_tool_success

1.1 ms

test_install_tool_failure

tests/test_util.py::test_install_tool_failure

1.4 ms

test_version_lists_not_empty

tests/test_util.py::test_version_lists_not_empty

110.5 µs

110.8 µs

test_get_runtime_version_clang_tidy

tests/test_util.py::test_get_runtime_version_clang_tidy

512.1 µs

514 µs

test_get_version_from_dependency_missing_file

tests/test_util.py::test_get_version_from_dependency_missing_file

945.9 µs

950 µs

test_resolve_version_clang_tidy[None-None]

tests/test_util.py::test_resolve_version_clang_tidy[None-None]

-1%

85.8 µs

86.4 µs

test_default_versions

tests/test_util.py::test_default_versions

-1%

83.6 µs

84.3 µs

test_resolve_version_clang_format[None-None]

tests/test_util.py::test_resolve_version_clang_format[None-None]

-1%

86.1 µs

86.9 µs

Commits

Click on a commit to change the comparison range

Base

main

90a453d

-60%

fix: update tests

ba073f7

24 days ago

by shenxianpeng

fix: update run.sh

0a4c0ba

24 days ago

by shenxianpeng

Home Terms Privacy Docs