Merge pull request #73 from TrickfireRobotics/vlad/masterurc #67
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
name: Lint | |
on: [push, pull_request] | |
jobs: | |
lint: | |
runs-on: ubuntu-latest | |
steps: | |
- uses: actions/checkout@v4 | |
# Black has a first-party ci/cd solution doesn't require python or setup, so use that | |
# before running setup. | |
- name: Run Black | |
uses: psf/black@stable | |
with: | |
options: "--check --verbose" | |
src: "./src" | |
version: 24.8.0 | |
- name: Setup Python | |
uses: actions/setup-python@v5 | |
with: | |
python-version: 3.10.12 | |
- name: Install Dependencies | |
run: | | |
python -m pip install --upgrade pip | |
pip install pylint==3.2.7 mypy==1.11.2 moteus | |
- name: Run Pylint | |
# Ignoring ros packages is unideal, but installing them as packages is not trivial | |
run: | | |
pylint --fail-under=8 --ignored-modules=rclpy,std_msgs,ament_copyright,ament_flake8,pytest,launch ./src | |
- name: Run Mypy | |
run: | | |
mypy --exclude /test/ --exclude setup\.py ./src |