Skip to content

Conversation

@mcbarton
Copy link
Collaborator

@mcbarton mcbarton commented Mar 26, 2025

Description

Please include a summary of changes, motivation and context for this PR.

This PR removes trailing whitespace in workflow yml files. This fixes one of the issues needed to have a yaml file linter check any workflow file changes in the future. Currently such a workflow would not pass.

I've also used this PR to remove the trailing whitespaces in the source files, test files and cmakelists.txt files

Fixes # (issue)

Type of change

Please tick all options which are relevant.

  • Bug fix
  • New feature
  • Requires documentation updates

Testing

Please describe the test(s) that you added and ran to verify your changes.

Checklist

  • I have read the contribution guide recently

@mcbarton mcbarton changed the base branch from whitespace-removal to main March 26, 2025 11:50
@mcbarton mcbarton force-pushed the whitespace-removal branch from 748e6ad to 5150624 Compare March 26, 2025 11:52
@codecov
Copy link

codecov bot commented Mar 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.46%. Comparing base (10f5449) to head (1ee6dac).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #539   +/-   ##
=======================================
  Coverage   75.46%   75.46%           
=======================================
  Files           9        9           
  Lines        3628     3628           
=======================================
  Hits         2738     2738           
  Misses        890      890           
Files with missing lines Coverage Δ
include/clang/Interpreter/CppInterOp.h 96.29% <ø> (ø)
lib/Interpreter/DynamicLibraryManagerSymbol.cpp 68.53% <ø> (ø)
Files with missing lines Coverage Δ
include/clang/Interpreter/CppInterOp.h 96.29% <ø> (ø)
lib/Interpreter/DynamicLibraryManagerSymbol.cpp 68.53% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mcbarton mcbarton changed the title [ci] Remove trailing whitespace in workflow yml files [ci] Remove trailing whitespace in workflow yml files, source files, test files and cmakelists.txt files Mar 26, 2025
@mcbarton mcbarton force-pushed the whitespace-removal branch from 67f0290 to 1ee6dac Compare March 26, 2025 12:29
@github-actions
Copy link
Contributor

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

@mcbarton mcbarton merged commit a9a865e into compiler-research:main Mar 26, 2025
73 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.

1 participant