diff mbox series

ci/pylama: Added pylama config file

Message ID 20210812162255.80127-1-ohilyard@iol.unh.edu (mailing list archive)
State New
Headers show
Series ci/pylama: Added pylama config file | expand

Commit Message

Owen Hilyard Aug. 12, 2021, 4:22 p.m. UTC
From: Owen Hilyard <ohilyard@iol.unh.edu>

I ended up turning off most of the formatting-releated checks due to
some minor conflicts with black (black allows lines > 80 characters for
instance). This produces a very substantial list of errors, some of
which may need to be simply disabled instead of being fixed due to
architectural reasons.

Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
---
 ci/requirements.txt | 5 ++++-
 pylama.ini          | 5 +++++
 2 files changed, 9 insertions(+), 1 deletion(-)
 create mode 100644 pylama.ini

Comments

Owen Hilyard Aug. 12, 2021, 4:29 p.m. UTC | #1
This depends on the requirements file that I added when adding the other ci
script. I want to make it as easy as possible to install everything needed
for CI, which means all of it should go in one requirements file. Even with
my removing many issues, this still has a lot of issues with the DTS
codebase, 35126 issues to be exact. We may want to bring this back to the
working group and discuss more, after fixing just the errors, since there
is a manageable amount of those.

On Thu, Aug 12, 2021 at 12:23 PM <ohilyard@iol.unh.edu> wrote:

> From: Owen Hilyard <ohilyard@iol.unh.edu>
>
> I ended up turning off most of the formatting-releated checks due to
> some minor conflicts with black (black allows lines > 80 characters for
> instance). This produces a very substantial list of errors, some of
> which may need to be simply disabled instead of being fixed due to
> architectural reasons.
>
> Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
> ---
>  ci/requirements.txt | 5 ++++-
>  pylama.ini          | 5 +++++
>  2 files changed, 9 insertions(+), 1 deletion(-)
>  create mode 100644 pylama.ini
>
> diff --git a/ci/requirements.txt b/ci/requirements.txt
> index 0abd6716..e19e540e 100644
> --- a/ci/requirements.txt
> +++ b/ci/requirements.txt
> @@ -29,4 +29,7 @@
>  # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
>  # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>
> -argparse==1.4.0
> \ No newline at end of file
> +argparse==1.4.0
> +pylama
> +pylama_pylint
> +black
> \ No newline at end of file
> diff --git a/pylama.ini b/pylama.ini
> new file mode 100644
> index 00000000..aef408c0
> --- /dev/null
> +++ b/pylama.ini
> @@ -0,0 +1,5 @@
> +[pylama]
> +format = pylint
> +linters = pep8,pycodestyle,pylint
> +ignore = F0401,C0111,E731,E266,E501,E203
> +
> --
> 2.30.2
>
>
diff mbox series

Patch

diff --git a/ci/requirements.txt b/ci/requirements.txt
index 0abd6716..e19e540e 100644
--- a/ci/requirements.txt
+++ b/ci/requirements.txt
@@ -29,4 +29,7 @@ 
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-argparse==1.4.0
\ No newline at end of file
+argparse==1.4.0
+pylama
+pylama_pylint
+black
\ No newline at end of file
diff --git a/pylama.ini b/pylama.ini
new file mode 100644
index 00000000..aef408c0
--- /dev/null
+++ b/pylama.ini
@@ -0,0 +1,5 @@ 
+[pylama]
+format = pylint
+linters = pep8,pycodestyle,pylint
+ignore = F0401,C0111,E731,E266,E501,E203
+