Skip to content

Conversation

@pasabanov
Copy link
Member

@pasabanov pasabanov commented Mar 31, 2025

Types of changes

  • Configuration (CI/CD)

Related: #46 #58

Description

Clang-Tidy started failing due to the -Wno-lto-type-mismatch and -s options for the Google Benchmark object files.
This is strange, as it didn't fail right after the benchmarks were implemented, and they haven't been changed since, especially especially considering Clang-Tidy should ignore the source files for these object files.

This commit adds the -Wno-unknown-warning-option and -Wno-unused-command-line-argument options to Clang-Tidy.

Additional information

  1. Logs from commit bb771dd (bench: Introduced benchmarks (bench: Introduced benchmarks #58)):
    logs_36137402605.zip
  2. Logs from commit cab8b26 (bench(spline): Implemented benchmarks for PolyEqvSpline (bench(spline): Implemented benchmarks for PolyEqvSpline #63)):
    logs_36402007344.zip
  3. Logs from the same commit cab8b26, but from a different repository and 2 days later:
    logs_36439522113.zip

@pasabanov pasabanov added the config Configuring the project label Mar 31, 2025
@pasabanov pasabanov added this to the First Release Version milestone Mar 31, 2025
@pasabanov pasabanov self-assigned this Mar 31, 2025
@pasabanov pasabanov force-pushed the ci branch 2 times, most recently from a70bbbf to 18f4632 Compare March 31, 2025 02:04
Clang-Tidy started failing due to the `-Wno-lto-type-mismatch` and `-s` options for the Google Benchmark object files.
This is strange, as it didn't fail right after the benchmarks were implemented, and they haven't been changed since,
especially considering Clang-Tidy should ignore the source files for these object files.

This commit adds the `-Wno-unknown-warning-option` and `-Wno-unused-command-line-argument` options to Clang-Tidy.
@pasabanov pasabanov merged commit bfda096 into main Mar 31, 2025
7 checks passed
@pasabanov pasabanov deleted the ci branch March 31, 2025 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Configuring the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants