obsproject / obs-plugintemplate

GNU General Public License v2.0
285 stars 133 forks source link

Update clang-format to 16 #110

Closed EZ64cool closed 8 months ago

EZ64cool commented 8 months ago

Description

Upgrade clang-format used in actions to 16

Motivation and Context

With changes introduced in https://github.com/obsproject/obs-plugintemplate/pull/82 clang-format 16 is now required. Using the current version of 13 will result in a .clang-format:115:3: error: unknown enumerated scalar error

How Has This Been Tested?

I have made small edits to code files which should trigger the github action which I will then revert. I have also implemented this change into my own plugin after merging latest. https://github.com/EZ64cool/obs-hadowplay/actions/workflows/pr-pull.yaml?query=branch%3Amerging_template

Types of changes

Checklist:

PatTheMav commented 8 months ago

Commits need to be squashed and the experimental change reverted before that.

RytoEX commented 8 months ago

Commits need to be squashed and the experimental change reverted before that.

Ah, I didn't see the commits. Yes, this feedback needs to be addressed.

EZ64cool commented 8 months ago

Thanks, first time contributing so sorry for the mess. I'll make a seperate FQ for running clang-format on other relevant files to keep this neat.