Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Static analysers in CI #329

Merged
merged 1 commit into from
May 11, 2023
Merged

Static analysers in CI #329

merged 1 commit into from
May 11, 2023

Conversation

Meakk
Copy link
Contributor

@Meakk Meakk commented Jul 2, 2022

No description provided.

@Meakk Meakk self-assigned this Jul 2, 2022
@codecov
Copy link

codecov bot commented Jul 2, 2022

Codecov Report

Merging #329 (596a945) into master (06b2129) will decrease coverage by 0.01%.
The diff coverage is 94.36%.

@@            Coverage Diff             @@
##           master     #329      +/-   ##
==========================================
- Coverage   96.09%   96.09%   -0.01%     
==========================================
  Files         109      109              
  Lines        6172     6166       -6     
==========================================
- Hits         5931     5925       -6     
  Misses        241      241              
Impacted Files Coverage Δ
application/F3DSystemTools.cxx 72.22% <0.00%> (ø)
...ry/VTKExtensions/Applicative/vtkF3DObjectFactory.h 66.66% <ø> (ø)
...ary/VTKExtensions/Core/vtkF3DConsoleOutputWindow.h 100.00% <ø> (ø)
...rary/VTKExtensions/Readers/vtkF3DGenericImporter.h 100.00% <ø> (ø)
...ry/VTKExtensions/Readers/vtkF3DPostProcessFilter.h 100.00% <ø> (ø)
...tensions/Rendering/vtkF3DInteractorEventRecorder.h 100.00% <ø> (ø)
...ary/VTKExtensions/Rendering/vtkF3DNoRenderWindow.h 100.00% <ø> (ø)
library/VTKExtensions/Rendering/vtkF3DRenderPass.h 100.00% <ø> (ø)
...KExtensions/Rendering/vtkF3DRendererWithColoring.h 100.00% <ø> (ø)
library/private/animationManager.h 100.00% <ø> (ø)
... and 23 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Meakk Meakk force-pushed the clang-tidy-pass branch 3 times, most recently from 160d807 to 5353894 Compare July 2, 2022 18:06
@Meakk Meakk requested a review from mwestphal July 2, 2022 18:07
library/public/image.h Outdated Show resolved Hide resolved
@mwestphal
Copy link
Contributor

Nice, but adding a tidy job (see #203 ) seems necessary to prevent this.

.clang-tidy Outdated Show resolved Hide resolved
@Meakk Meakk force-pushed the clang-tidy-pass branch 7 times, most recently from 12ef8e5 to e9d1daa Compare July 5, 2022 10:48
@mwestphal mwestphal mentioned this pull request Aug 14, 2022
14 tasks
@Meakk Meakk force-pushed the clang-tidy-pass branch 2 times, most recently from 5c5c6d3 to 72dbb36 Compare April 7, 2023 13:53
@Meakk
Copy link
Contributor Author

Meakk commented Apr 14, 2023

I decided to try SonarCloud instead.

@Meakk Meakk force-pushed the clang-tidy-pass branch 7 times, most recently from bb1b07e to 50e50f8 Compare May 9, 2023 16:05
@Meakk Meakk force-pushed the clang-tidy-pass branch 3 times, most recently from 4ed6ae6 to 8e6e5c0 Compare May 10, 2023 09:19
@github-actions
Copy link

You are modifying libf3d public API! ⚠️Please update bindings accordingly⚠️!
You can find them in their respective directories: python, java, webassembly.

@Meakk Meakk force-pushed the clang-tidy-pass branch 9 times, most recently from a0cf010 to ee12645 Compare May 10, 2023 21:52
@Meakk Meakk changed the title clang-tidy pass Static analysers in CI May 10, 2023
.clang-tidy Show resolved Hide resolved
.cppcheck.supp Outdated Show resolved Hide resolved
@Meakk Meakk merged commit de0d8e9 into f3d-app:master May 11, 2023
@Meakk Meakk deleted the clang-tidy-pass branch May 11, 2023 14:36
mwestphal pushed a commit that referenced this pull request Feb 10, 2024
mwestphalnew pushed a commit to mwestphalnew/f3d that referenced this pull request Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants