Merge 'develop' into tst/flight-class-protection

Author: Gui-FernandesBR

.github/auto-assign.yml

@@ -4,16 +4,19 @@ addReviewers: true
 # Set to 'author' to add PR's author as a assignee
 addAssignees: author
 
-# A list of reviewers to be added to PRs (GitHub user  name)
+# A list of reviewers to be added to PRs (GitHub user name)
 reviewers:
   - Gui-FernandesBR
   - giovaniceotto
   - MateusStano
-  
+  - phmbressan
+
 # A number of reviewers added to the PR
 # Set 0 to add all the reviewers (default: 0)
 numberOfReviewers: 0
 
 # A list of keywords to be skipped the process if PR's title include it
 skipKeywords:
   - wip
+  - work in progress
+  - draft

.github/pull_request_template.md

@@ -4,52 +4,35 @@
 
 ## Pull request type
 
-Please check the type of change your PR introduces:
-
-- [ ] Code base additions (bugfix, features)
-- [ ] Code maintenance (refactoring, formatting, renaming, tests)
-- [ ] ReadMe, Docs and GitHub maintenance
+- [ ] Code changes (bugfix, features)
+- [ ] Code maintenance (refactoring, formatting, tests)
+- [ ] ReadMe, Docs and GitHub updates
 - [ ] Other (please describe):
 
-## Pull request checklist
-
-Please check if your PR fulfills the following requirements, depending on the type of PR:
-
-- ReadMe, Docs and GitHub maintenance:
-
-  - [ ] Spelling has been verified
-  - [ ] Code docs are working correctly 
+## Checklist
 
-- Code base maintenance (refactoring, formatting, renaming):
+- [ ] Tests for the changes have been added (if needed)
+- [ ] Docs have been reviewed and added / updated
+- [ ] Lint (`black rocketpy/ tests/`) has passed locally 
+- [ ] All tests (`pytest --runslow`) have passed locally
 
-  - [ ] Docs have been reviewed and added / updated if needed
-  - [ ] Lint (`black rocketpy`) has passed locally and any fixes were made
-  - [ ] All tests (`pytest --runslow`) have passed locally
-
-- Code base additions (for bug fixes / features):
-
-  - [ ] Tests for the changes have been added
-  - [ ] Docs have been reviewed and added / updated if needed
-  - [ ] Lint (`black rocketpy`) has passed locally and any fixes were made
-  - [ ] All tests (`pytest --runslow`) have passed locally
-
-## What is the current behavior?
-<!-- Please describe the current behavior that you are modifying, or link to a relevant issue. -->
+## Current behavior
+<!-- Describe current behavior or link to an issue. -->
 
 Enter text here...
 
-## What is the new behavior?
-<!-- Please describe the behavior or changes that are being added by this PR. -->
+## New behavior
+<!-- Describe changes introduced by this PR. -->
 
 Enter text here...
 
-## Does this introduce a breaking change?
+## Breaking change
 <!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
 
 - [ ] Yes
 - [ ] No
 
-## Other information
-<!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
+## Additional information
+<!-- Include any relevant details or screenshots. -->
 
-Enter text here...
+Enter text here...

.github/workflows/auto-assign-projects

@@ -1,7 +1,5 @@
-name: Auto Assign Issues and PRs once opened
+name: Auto Assign PRs once opened
 on:
-  issues:
-    types: [opened]
   pull_request:
     types: [opened]
 jobs:

.github/workflows/auto-assign.yml

@@ -1,9 +1,19 @@
 name: PyTest
 
-on: [pull_request]
+on:
+  pull_request:
+    types: [opened, synchronize, reopened, ready_for_review]
+    paths:
+      - "**.py"
 
 jobs:
-  build:
+  fail_if_pull_request_is_draft:
+    if: github.event.pull_request.draft == true
+    runs-on: ubuntu-18.04
+    steps:
+      - name: Fails in order to indicate that pull request needs to be marked as ready to review and unit tests workflow needs to pass.
+        run: exit 1
+  run_pytest_and_doctest:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
@@ -23,10 +33,15 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install -r requirements-tests.txt
-      - name: Build RocketPy
+      - name: Build RocketPy (without optional dependencies)
+        run: |
+          pip install .
+      - name: Import rocketpy in python and test if it works
         run: |
-          pip install -e.[all]
+          python -c "import sys, rocketpy; print(f'{rocketpy.__name__} running on Python {sys.version}')"
+      - name: Install optional dependencies
+        run: |
+          pip install -r requirements-tests.txt
       - name: Test with pytest
         run: |
           pytest

.github/workflows/test_pytest.yaml

@@ -13,10 +13,14 @@ build:
 
 # Build documentation in the docs/ directory with Sphinx
 sphinx:
-   configuration: docs/conf.py
+  configuration: docs/conf.py
 
 # Optionally declare the Python requirements required to build your docs
 python:
-   install:
-   - requirements: docs/requirements.txt
-   - requirements: requirements.txt
+  install:
+    - requirements: docs/requirements.txt
+    - requirements: requirements.txt
+    - method: pip
+      path: .
+      extra_requirements:
+        - all