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.

Sign up for GitHub

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

Jump to bottom

ladislas/feature/add std span #269

Merged
merged 3 commits into from
Jun 3, 2021
Merged

ladislas/feature/add std span #269

merged 3 commits into from
Jun 3, 2021

Conversation

ladislas
Copy link
Member

@ladislas ladislas commented Jun 3, 2021

  • 🎨 (clangformat): Move clang-format off before explanation comment
  • ✨ (cxxsupport): Add lstd_span header
  • πŸš€ (spikes): Add stl_cxxsupport spike

@codecov
Copy link

codecov bot commented Jun 3, 2021 •
edited
Loading

Codecov Report

Merging #269 (1f7c8fe) into develop (b5e5af4) will not change coverage.
The diff coverage is n/a.

❗ Current head 1f7c8fe differs from pull request most recent head 1360e03. Consider uploading reports for the commit 1360e03 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##           develop      #269   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           56        56           
  Lines          936       936           
=========================================
  Hits           936       936

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update b5e5af4...1360e03. Read the comment docs.

YannLocatelli
YannLocatelli approved these changes Jun 3, 2021
View reviewed changes
spikes/stl_cxxsupport/CMakeLists.txt Outdated Show resolved Hide resolved
@ladislas ladislas force-pushed the ladislas/feature/add-std-span branch 3 times, most recently from 746aa9e to d9105ea Compare June 3, 2021 14:24
@codeclimate
Copy link

codeclimate bot commented Jun 3, 2021

Code Climate has analyzed commit d9105ea and detected 0 issues on this pull request.

View more on Code Climate.

@ladislas ladislas force-pushed the ladislas/feature/add-std-span branch from d9105ea to 1f7c8fe Compare June 3, 2021 15:19
@ladislas ladislas force-pushed the ladislas/feature/add-std-span branch from 1f7c8fe to 1360e03 Compare June 3, 2021 15:26
@sonarcloud
Copy link

sonarcloud bot commented Jun 3, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ladislas ladislas merged commit 7c973b4 into develop Jun 3, 2021
@ladislas ladislas deleted the ladislas/feature/add-std-span branch July 12, 2021 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YannLocatelli YannLocatelli YannLocatelli approved these changes

@Kabroc Kabroc Awaiting requested review from Kabroc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ladislas @YannLocatelli