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

[DO NOT MERGE] Test reviewdog #53

Closed
wants to merge 4 commits into from

Conversation

wusatosi
Copy link
Member

#46 introduced reviewdog but was having problem (#48).
#49 attempted to fix those problems. But not sure if that fixes it due to permission complications.

Unfortunately there's not a good way to test GitHub Actions and reviewdog, so I have to create this pr to see if they are working correctly.

Sorry for spamming the notification.

Expected:

  1. pre-commit flags the clang-format and white space linting issue and generates fixes
  2. Review dog suggests fixes via Pull Request reviews.

@wusatosi
Copy link
Member Author

wusatosi commented Oct 19, 2024

Permission issue persists.

Please forward discussion to #49 as this involves GitHub project setting.

Comment on lines +4 to +5
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey

Copy link
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey
* @bretbrownjr @camio @dietmarkuehl @neatudarius @steve-downey

Comment on lines +38 to +41
int
size
=
0;
Copy link
Contributor

Choose a reason for hiding this comment

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

[pre-commit] reported by reviewdog 🐶

Suggested change
int
size
=
0;
int size = 0;

@wusatosi
Copy link
Member Author

@neatudarius review dog is working now!!! YEY!!

@wusatosi wusatosi closed this Nov 10, 2024
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