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

Setup pre-commit to GEF-Extras #91

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Setup pre-commit to GEF-Extras #91

merged 5 commits into from
Aug 9, 2023

Conversation

hugsy
Copy link
Owner

@hugsy hugsy commented Aug 8, 2023

Description

Replicates GEF's pre-commit policy and apply it.

It also fixes all the minor formatting failures found by the policy.

How Has This Been Tested ?

"Tested" indicates that the PR works and the unit test (i.e. make test) run passes without issue.

N/A

Checklist

  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • If my change adds new code, adequate tests have been added.
  • I have read and agree to the CONTRIBUTING document.

@hugsy hugsy added this to the 2023.08 milestone Aug 8, 2023
@hugsy hugsy self-assigned this Aug 8, 2023
@hugsy hugsy merged commit 652e0f2 into main Aug 9, 2023
7 checks passed
@hugsy hugsy deleted the use_precommit branch August 9, 2023 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants