added Python test matrix (#42)

protectai · Apr 28, 2023 · fadd739 · fadd739
1 parent fa3afc3
commit fadd739
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -9,11 +9,16 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.7", "3.8", "3.9", "3.10"]
+
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
         with:
-          python-version: "3.8"
+          python-version: ${{ matrix.python-version }}
       - uses: snok/install-poetry@v1
         with:
           virtualenvs-create: true

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -40,8 +40,9 @@ If you're looking for documentation on how to use NB Defense, you can find that
 
 Thanks for contributing! In order to open a PR into the NB Defense project, you'll have to follow these steps:
 
-1. Fork the repo
-2. Make your changes
-3. Submit a pull request
+1. Fork the repo and clone your fork locally
+2. Run `make install-dev` from the root of your forked repo to setup your environment
+3. Make your changes
+4. Submit a pull request
 
 After these steps have been completed, someone on our team at Protect AI will review the code and help merge in your changes!