Merge pull request #2 from chavarera/master

Updating my fork

Author: namanshah01

.github/ISSUE_TEMPLATE.md

@@ -4,15 +4,15 @@
 
 ## Type of issue
 
-- [ ] Feature(New Script)
+- [ ] Feature (New Script)
 - [ ] Bug
 - [ ] Documentation
 
 ## Checklist:
 
 - [ ] I have read the project guidelines.
-- [ ] I have checked all the existing projects before submitting a new project issue
-- [ ] I have checked previous issues to avoid duplicates
+- [ ] I have checked all the existing projects, before submitting a new project issue.
+- [ ] I have checked previous issues to avoid duplicates.
 - [ ] This issue will be meaningful for the project.
 
 

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,10 +1,10 @@
 # Description
 
-A short summary of what is included in your pull request
+A short summary of what is included in your Pull Request.
 
 Fixes #(issue_no)
 
-Replace `issue_no` in the above line with the issue related to this PR.
+Replace `issue_no` in the above line, with the issue related to this PR.
 
 ## Type of change
 
@@ -15,13 +15,13 @@ Choosing one or more options from the following as per the nature of your Pull r
 - [ ] Documentation Update
 
 # Checklist:
-
-Please tick all the boxes that are fulfilled by your Pull request
-
-- [ ] I have named my files and folder according to this project's guidelines
-- [ ] My code follows the style guidelines of this project
-- [ ] I have commented my code, particularly in hard-to-understand areas
-- [ ] I have created a helpful and easy to understand `README.md`
-- [ ] I have included a requirements.txt file(if external libraries are required)
-- [ ] My changes dont produce any warnings.
-- [ ] I have added a working sample/screenshot of the script
+Please tick all the boxes that are fulfilled by your Pull Request.
+
+- [ ] I have named my files and folder, according to this project's guidelines.
+- [ ] My code follows the style guidelines of this project.
+- [ ] My Pull Request has a descriptive title. (not a vague title like `Update index.md`)
+- [ ] I have commented on my code, particularly in hard-to-understand areas.
+- [ ] I have created a helpful and easy to understand `README.md`, according to the given [`README_TEMPLATE.`](https://github.com/chavarera/python-mini-projects/blob/master/README_TEMPLATE.md)
+- [ ] I have included a requirements.txt file (if external libraries are required.)
+- [ ] My changes do not produce any warnings.
+- [ ] I have added a working sample/screenshot of the script.

.github/workflows/python-app.yml

@@ -0,0 +1,34 @@
+# This workflow will install Python dependencies, run tests and lint with a single version of Python
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions
+
+name: Python application
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Python 3.8
+      uses: actions/setup-python@v2
+      with:
+        python-version: 3.8
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install flake8 
+        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+    - name: Lint with flake8
+      run: |
+        # stop the build if there are Python syntax errors or undefined names
+        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+