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

Run linters across test folder. #3982

Merged
merged 3 commits into from
Apr 28, 2023
Merged

Run linters across test folder. #3982

merged 3 commits into from
Apr 28, 2023

Conversation

fruffy
Copy link
Collaborator

@fruffy fruffy commented Apr 17, 2023

The test folder was not checked. Also add it to the linted files.

@fruffy fruffy force-pushed the fruffy/run_clang_format_across_test branch 2 times, most recently from 3bd3598 to b4c57df Compare April 19, 2023 13:07
jnfoster
jnfoster approved these changes Apr 28, 2023
View reviewed changes
Copy link
Contributor

@jnfoster jnfoster left a comment

Choose a reason for hiding this comment

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

I did not review the entire PR carefully. Based on the comment and the nature of the changes, I assume most of the code changes are linter-induced.

@fruffy fruffy force-pushed the fruffy/run_clang_format_across_test branch from b4c57df to 06c369e Compare April 28, 2023 12:29
@fruffy fruffy merged commit 30d953b into main Apr 28, 2023
@fruffy fruffy deleted the fruffy/run_clang_format_across_test branch April 28, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jnfoster jnfoster jnfoster approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@fruffy @jnfoster