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

Add containerized pre-commit hook #191

Merged
merged 4 commits into from
Oct 21, 2023

Conversation

lhriley
Copy link
Contributor

@lhriley lhriley commented Aug 15, 2023

Simple PR that adds support for using the pre-build Docker container image rather than requiring users to install the binary or build the golang package from source.

Tested locally and works with this .pre-commit-config.yaml:

---
repos:
  - repo: https://github.com/Labelbox/helm-docs
    rev:  65f1d83 # add-docker-to-pre-commit-hooks
    hooks:
      - id: helm-docs-container
        args:
          # Make the tool search for charts only under the `charts` directory
          - --chart-search-root=charts

Related:
pre-commit/pre-commit#2968

@Nepo26
Copy link
Collaborator

Nepo26 commented Aug 29, 2023

Hm seems nice. Will look into it by the end of the next week.

@Nepo26 Nepo26 self-requested a review September 10, 2023 03:34
@Nepo26 Nepo26 self-assigned this Sep 10, 2023
@Nepo26 Nepo26 added this to the General Cleanup milestone Sep 10, 2023
@Nepo26 Nepo26 added the enhancement New feature or request label Sep 10, 2023
.pre-commit-hooks.yaml Outdated Show resolved Hide resolved
@Nepo26
Copy link
Collaborator

Nepo26 commented Sep 10, 2023

If possible also add to the README the description of how to define the use of the container and how it should be ran.

lhriley and others added 2 commits October 16, 2023 14:28
Co-authored-by: Henrique Nepo <me@henriquenepomuceno.com>
@lhriley
Copy link
Contributor Author

lhriley commented Oct 16, 2023

@Nepo26 I pushed up changes to the README as requested. There were some whitespace issues that were cleaned up by my editor that makes the diff a bit messy.

@Nepo26
Copy link
Collaborator

Nepo26 commented Oct 21, 2023

Just tested here and didn't see any problem.

Thanks @lhriley - looks great :)

@Nepo26 Nepo26 closed this Oct 21, 2023
@Nepo26 Nepo26 reopened this Oct 21, 2023
@Nepo26 Nepo26 merged commit eadcfd7 into norwoodj:master Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants