Skip to content

Conversation

mcbarton
Copy link
Collaborator

Description

Please include a summary of changes, motivation and context for this PR.
This PR will attempt to fix the clang tidy workflow.

Fixes # (issue)

Type of change

Please tick all options which are relevant.

  • Bug fix
  • New feature
  • Added/removed dependencies
  • Required documentation updates

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.83%. Comparing base (baba783) to head (3baf74c).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   81.83%   81.83%           
=======================================
  Files          19       19           
  Lines         837      837           
  Branches       91       91           
=======================================
  Hits          685      685           
  Misses        152      152           
Files with missing lines Coverage Δ
src/xinterpreter.cpp 88.75% <ø> (ø)
Files with missing lines Coverage Δ
src/xinterpreter.cpp 88.75% <ø> (ø)

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Contributor

@vgvassilev vgvassilev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@vgvassilev vgvassilev merged commit f678e0d into compiler-research:main Oct 16, 2024
9 of 10 checks passed
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.

3 participants