-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #39 from Sindri-Labs/kp-complete-method-params
Refactor internal methods, update docstrings, align methods to api endpoints
- Loading branch information
Showing
12 changed files
with
825 additions
and
398 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,52 @@ | ||
name: Lint Codebase | ||
|
||
on: | ||
push: | ||
branches: [ "main" ] | ||
pull_request: | ||
# Do not run on draft pull requests | ||
types: | ||
- opened | ||
- reopened | ||
- synchronize | ||
- ready_for_review | ||
|
||
jobs: | ||
run-lint: | ||
runs-on: ubuntu-latest | ||
if: ${{ github.event_name == 'push' || !github.event.pull_request.draft }} # Do not run on draft pull requests | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Python 3.10.6 | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: 3.10.6 | ||
|
||
- name: Install poetry | ||
run: pip install poetry | ||
|
||
- name: Install Dependencies | ||
run: poetry install | ||
|
||
- name: Check formatting with black | ||
if: success() || failure() | ||
run: | | ||
poetry run black --check . | ||
- name: Check import order with isort | ||
if: success() || failure() | ||
run: | | ||
poetry run isort --check-only . | ||
- name: Lint with flake8 | ||
if: success() || failure() | ||
run: | | ||
poetry run flake8 . | ||
- name: Verify typing annotations | ||
if: success() || failure() | ||
run: | | ||
poetry run mypy . |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,3 +11,5 @@ skip = | |
.pytest_cache/* | ||
.vscode | ||
__pycache__/* | ||
venv | ||
env |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
#!/bin/bash | ||
|
||
# File formatters | ||
echo "" | ||
echo "(python) black..." | ||
black . | ||
|
||
|
||
echo "" | ||
echo "(python) isort..." | ||
isort --overwrite-in-place . | ||
|
||
# Analyze code and report errors | ||
echo "" | ||
echo "(python) flake8..." | ||
flake8 . | ||
|
||
echo "" | ||
echo "(python) mypy..." | ||
mypy . | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Empty file.
Oops, something went wrong.