cisagov / skeleton-ansible-role

A skeleton project for quickly getting a new cisagov Ansible role started.
Creative Commons Zero v1.0 Universal
7 stars 6 forks source link

Add a lower-bound pin for the `pip` package `flake8-docstrings` #204

Closed jsf9k closed 4 months ago

jsf9k commented 4 months ago

๐Ÿ—ฃ Description

I noticed that flake8-docstrings is listed as an additional dependency of flake8 in the pre-commit configuration, but pinning is done.

๐Ÿ’ญ Motivation and context

@mcdonnnj agreed with me that it makes sense to at least perform some minimal pinning to ensure that a working version of flake8-docstrings is installed.

๐Ÿงช Testing

All automated tests pass. I also verified that there are no other additional dependencies in the pre-commit configuration that do no at least have a lower-bound pin.

โœ… Pre-approval checklist

jsf9k commented 4 months ago

Wrong repo!