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

918 update ci #945

Merged
merged 33 commits into from
Feb 21, 2023
Merged

918 update ci #945

merged 33 commits into from
Feb 21, 2023

Conversation

AntonReiswich
Copy link
Contributor

Closes #918

@joergbrech joergbrech force-pushed the 918-Update-CI branch 2 times, most recently from 3ade0d5 to 11fb948 Compare February 17, 2023 16:29
@joergbrech
Copy link
Contributor

I think we've got it! @AntonReiswich, would you mind having a quick glimpse at my changes and merge, if you approve and CI finishes up successfully?

@codecov-commenter
Copy link

Codecov Report

Merging #945 (24da825) into master (dc973ee) will decrease coverage by 0.81%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #945      +/-   ##
==========================================
- Coverage   69.56%   68.76%   -0.81%     
==========================================
  Files         298      299       +1     
  Lines       25014    26483    +1469     
==========================================
+ Hits        17401    18210     +809     
- Misses       7613     8273     +660     
Impacted Files Coverage Δ
src/common/typename.cpp 71.42% <0.00%> (-11.91%) ⬇️
src/common/stringtools.h 87.09% <0.00%> (-5.77%) ⬇️
src/cpacs_other/CTiglUIDManager.h 95.83% <0.00%> (-4.17%) ⬇️
src/exports/CTiglCADExporter.cpp 60.00% <0.00%> (-3.81%) ⬇️
src/api/tigl.cpp 51.68% <0.00%> (-3.44%) ⬇️
src/wing/CTiglWingConnection.cpp 82.97% <0.00%> (-3.39%) ⬇️
src/wing/CCPACSWingCells.cpp 60.00% <0.00%> (-3.16%) ⬇️
src/structural_elements/CCPACSSheetList.cpp 20.00% <0.00%> (-3.08%) ⬇️
src/geometry/CTiglInterpolatePointsWithKinks.cpp 97.36% <0.00%> (-2.64%) ⬇️
src/wing/CCPACSWingProfiles.cpp 70.96% <0.00%> (-2.37%) ⬇️
... and 150 more

@AntonReiswich
Copy link
Contributor Author

That is great @joergbrech, thanks a lot! Sure, I'll look through it now!

@AntonReiswich AntonReiswich merged commit 900c4f5 into master Feb 21, 2023
@AntonReiswich AntonReiswich deleted the 918-Update-CI branch February 21, 2023 08:29
@joergbrech
Copy link
Contributor

MacOS tests still seam to be flakey (this isn't a new issue, we had that problem before)...

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.

Update CI
3 participants