Skip to content

Conversation

@GOB52
Copy link
Contributor

@GOB52 GOB52 commented Aug 22, 2024

  • Added the condition that the file must be a file with the target extension, since it works unconditionally regardless of the file type.
    For example, it also works when a .txt file is modified.
    This will result in unnecessary Actions being run.
  • Raise checkout version from v3 to v4.
    Use node20 editions
  • Raise clang-format-action from v4.9 0 to v4.10.2.
    Need to upgrade to use the option to increase target file extensions

M5UnitUnified related uses this as a basis.

Please consider it.

@imliubo
Copy link
Member

imliubo commented Aug 22, 2024

Thx.

@imliubo imliubo merged commit c41ce39 into m5stack:main Aug 22, 2024
@GOB52 GOB52 deleted the clang-format-action branch August 22, 2024 07:47
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.

2 participants