singingwolfboy / flask-dance

Doing the OAuth dance with style using Flask, requests, and oauthlib.
https://pypi.python.org/pypi/Flask-Dance/
MIT License
1.01k stars 158 forks source link

Simple way to mimic GH Actions checks #352

Open kerryhatcher opened 3 years ago

kerryhatcher commented 3 years ago

As a total NOOB contributor I would like a way to easily run all the tests and checks from one command in the root of the project so that I don't have to wait for the automated process to tell me I've forgotten something and have to rerun it.

Suggested fix: A main Makefile similar to the one in ./docs that invokes all the things.

Some of those things are:

Here is what threw together after my last "black" failure: https://gist.github.com/kerryhatcher/dba1959f2ea349268f308043895674d3. In that Gist I'm using pydocstyle as a way to quickly error out on docstring issues without having to wait around on sphinx-build. The output of pydocstyle is also cleaner since it only tests changes since last commit. That would have to be added in to the requirements.txt if that is to be adopted. More info on that here: https://github.com/PyCQA/pydocstyle/

Since I already have something I'd be happy to work on a PR but I thought we could flesh out the idea here a bit before jumping right into the code.

singingwolfboy commented 3 years ago

To be honest, I don't think I've ever written a Makefile in my life -- the one in the docs directory was auto-generated when I set up Sphinx, and I've never touched it. That being said, I like this idea!

My only concern is that the GitHub Actions workflows and the make workflows might diverge at some point in the future, creating more confusion. We could try to make GHA use make, which would prevent that, but might also make the GHA configs more complicated.

Frankly, I'm not too concerned; if things diverge in the future, then we can just re-align them. Maybe everything will be fine, and things will never diverge! At any rate, I think this is a good idea, and I would be happy to accept a pull request for a Makefile like the one you put in your gist.