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

Improve concept definitions #751

Merged
merged 3 commits into from
Jul 27, 2023

Conversation

bernhardmgruber
Copy link
Member

@bernhardmgruber bernhardmgruber commented Jul 21, 2023

This PR formats and improves some of the concept definitions in LLAMA.

@bernhardmgruber
Copy link
Member Author

bernhardmgruber commented Jul 21, 2023

The concept formatting looks way nicer with clang-format-16. I think we should update before merging this PR.

Edit: done in #752

@codecov
Copy link

codecov bot commented Jul 21, 2023

Codecov Report

Merging #751 (d034bcc) into develop (b376b17) will not change coverage.
The diff coverage is 100.00%.

❗ Current head d034bcc differs from pull request most recent head 04fc187. Consider uploading reports for the commit 04fc187 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #751   +/-   ##
========================================
  Coverage    98.80%   98.80%           
========================================
  Files           75       75           
  Lines         7212     7212           
========================================
  Hits          7126     7126           
  Misses          86       86           

@bernhardmgruber bernhardmgruber force-pushed the format_concepts branch 5 times, most recently from e7461fc to 51665ea Compare July 24, 2023 17:13
@bernhardmgruber bernhardmgruber marked this pull request as ready for review July 24, 2023 17:14
@bernhardmgruber bernhardmgruber merged commit ac3df87 into alpaka-group:develop Jul 27, 2023
31 of 36 checks passed
@bernhardmgruber bernhardmgruber deleted the format_concepts branch July 27, 2023 13:15
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.

1 participant