catalyst icon indicating copy to clipboard operation
catalyst copied to clipboard

Update the clang-format config file to ensure consistency with PL code style

Open maliasadi opened this issue 2 years ago • 2 comments

This PR includes following updates in the code-base,

  • Update the C++ clang-format config file to ensure consistency with PL code style introduced here.
  • Add a missing std library to the DataView class.
  • Add missing headers to a few dialects files.
  • Fix pylint warnings and useless-suppression

maliasadi avatar Oct 02 '23 18:10 maliasadi

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.51%. Comparing base (a0f7b93) to head (323f83e). Report is 577 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #295      +/-   ##
==========================================
- Coverage   99.51%   99.51%   -0.01%     
==========================================
  Files          40       40              
  Lines        7266     7248      -18     
  Branches      423      423              
==========================================
- Hits         7231     7213      -18     
  Misses         17       17              
  Partials       18       18              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 02 '23 19:10 codecov[bot]

Hello. You may have forgotten to update the changelog! Please edit doc/changelog.md on your branch with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

github-actions[bot] avatar Oct 03 '23 02:10 github-actions[bot]

I'll close this as stale, but we can reopen after a discussion about how we want to move forward with this.

dime10 avatar Jul 12 '24 15:07 dime10