You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PEP8Speaks needs to be hosted on a server to be run. A GitHub webhook sends payloads to the server via "GitHub app" and the server does all the magic. For users who want to run PEP8Speaks on a private repository, it is recommended to deploy a fork.
There can be an even easier distribution possible for PEP8Speaks. This is to run it on the repository's CI as a lint job. If the user has setup Travis/CircleCI/etc. on their repository (for building and testing their PRs), they should be able to add a job for PEP8Speaks.
PEP8Speaks needs to be converted into a Python package which generates a CLI pep8speaks. The CLI takes few parameters like GitHub token and Pull Request information, and it does exactly what PEP8Speaks should do (Comment on the PR by following .pep8speaks.yml).
The text was updated successfully, but these errors were encountered:
Was just browsing to see if I could set up a GitHub Actions workflow for pep8speaks.
Would be awesome if pep8speaks could be run as a CI job. Really looking forward to this feature.
PEP8Speaks needs to be hosted on a server to be run. A GitHub webhook sends payloads to the server via "GitHub app" and the server does all the magic. For users who want to run PEP8Speaks on a private repository, it is recommended to deploy a fork.
There can be an even easier distribution possible for PEP8Speaks. This is to run it on the repository's CI as a lint job. If the user has setup Travis/CircleCI/etc. on their repository (for building and testing their PRs), they should be able to add a job for PEP8Speaks.
PEP8Speaks needs to be converted into a Python package which generates a CLI
pep8speaks
. The CLI takes few parameters like GitHub token and Pull Request information, and it does exactly what PEP8Speaks should do (Comment on the PR by following.pep8speaks.yml
).The text was updated successfully, but these errors were encountered: