secureli icon indicating copy to clipboard operation
secureli copied to clipboard

Fine tuning base config

Open gabenegron opened this issue 2 years ago • 1 comments

Description

As of now, when secureli builds the .pre-commit.yaml file, it combines the elements from all language config files and a base-pre-commit.yaml file. The base pre commit file right now contains all hooks from the pre-commit/pre-commit repo and the secret detecting hook. This works, but some hooks may be unnecessarily running for specific languages.

Acceptance Criteria

Determine hooks that should be in base-pre-commit.yaml and what hooks should be moved to their language specific configs.

gabenegron avatar Jul 11 '23 19:07 gabenegron

Looking over the base-pre-commit.yaml I'm not sure any of the currently included hooks are language specific/agnostic. This may have been updated since this ticket was made almost a year ago. Happy to review again but this may be able to be closed.

JordoHeffernan avatar Mar 14 '24 16:03 JordoHeffernan