Merge pull request #514 from MITLibraries/timx-189-add-skip-list

Timx 189 add skip list

.dockerignore

@@ -1,5 +1,111 @@
-out.xml
+# Byte-compiled / optimized / DLL files
+__pycache__/
+*.py[cod]
+*$py.class
+
+# C extensions
+*.so
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+.hypothesis/
+.pytest_cache/
+coverage/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+target/
+
+# Jupyter Notebook
+.ipynb_checkpoints
+
+# pyenv
+# .python-version
+
+# celery beat schedule file
+celerybeat-schedule
+
+# SageMath parsed files
+*.sage.py
+
+# Environments
+.env
+.venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+
+# mypy
+.mypy_cache/
 tmp/*
-dist/*
-build/*
-*.egg-info
+out.xml
+.DS_Store
+.vscode
+
+output/

.github/dependabot.yml

@@ -1,82 +1,28 @@
+# To get started with Dependabot version updates, you'll need to specify which
+# package ecosystems to update and where the package manifests are located.
+# Please see the documentation for all configuration options:
+# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates
+
 version: 2
 updates:
-- package-ecosystem: pip
-  directory: "/"
-  schedule:
-    interval: daily
-    time: "09:00"
-    timezone: America/New_York
-  open-pull-requests-limit: 10
-  ignore:
-  - dependency-name: boto3
-    versions:
-    - 1.16.62
-    - 1.16.63
-    - 1.17.0
-    - 1.17.1
-    - 1.17.10
-    - 1.17.11
-    - 1.17.12
-    - 1.17.13
-    - 1.17.14
-    - 1.17.15
-    - 1.17.16
-    - 1.17.17
-    - 1.17.18
-    - 1.17.19
-    - 1.17.2
-    - 1.17.20
-    - 1.17.21
-    - 1.17.22
-    - 1.17.23
-    - 1.17.24
-    - 1.17.25
-    - 1.17.26
-    - 1.17.27
-    - 1.17.28
-    - 1.17.29
-    - 1.17.3
-    - 1.17.30
-    - 1.17.32
-    - 1.17.33
-    - 1.17.34
-    - 1.17.35
-    - 1.17.36
-    - 1.17.38
-    - 1.17.39
-    - 1.17.4
-    - 1.17.40
-    - 1.17.41
-    - 1.17.43
-    - 1.17.44
-    - 1.17.45
-    - 1.17.46
-    - 1.17.47
-    - 1.17.48
-    - 1.17.49
-    - 1.17.5
-    - 1.17.50
-    - 1.17.51
-    - 1.17.52
-    - 1.17.53
-    - 1.17.54
-    - 1.17.55
-    - 1.17.56
-    - 1.17.57
-    - 1.17.58
-    - 1.17.59
-    - 1.17.6
-    - 1.17.7
-    - 1.17.8
-    - 1.17.9
-  - dependency-name: flake8
-    versions:
-    - 3.8.4
-    - 3.9.0
-  - dependency-name: smart-open
-    versions:
-    - 4.1.2
-    - 4.2.0
-  - dependency-name: lxml
-    versions:
-    - 4.6.2
+  # Maintain dependencies for GitHub Actions
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "daily"
+
+  # Maintain dependencies for application
+  - package-ecosystem: "pip"
+    directory: "/"
+    schedule:
+      interval: "daily"
+    reviewers:
+      - "MITLibraries/dataeng"
+
+  # Maintain dependencies for Docker
+  - package-ecosystem: "docker"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    reviewers:
+      - "MITLibraries/dataeng"

.github/pull-request-template.md

@@ -1,28 +1,38 @@
+#### What does this PR do?
+
+Describe the overall purpose of the PR changes. Doesn't need to be as specific as the
+individual commits.
+
 #### Helpful background context
+
 Describe any additional context beyond what the PR accomplishes if it is likely
 to be useful to a reviewer.
 
 Delete this section if it isn't applicable to the PR.
 
 #### How can a reviewer manually see the effects of these changes?
+
 Explain how to see the proposed changes in the application if possible.
 
 Delete this section if it isn't applicable to the PR.
 
 #### Includes new or updated dependencies?
+
 YES | NO
 
 #### What are the relevant tickets?
+
 Include links to Jira Software and/or Jira Service Management tickets here.
 
-#### Developer
+#### Developer checklist
+
 - [ ] All new ENV is documented in README
 - [ ] Stakeholder approval has been confirmed (or is not needed)
 
-#### Code Reviewer
-- [ ] The commit message is clear and follows our guidelines
-      (not just this pull request message)
+#### Code reviewer checklist
+
+- [ ] The commit message is clear and follows our guidelines (not just this pull request message)
 - [ ] There are appropriate tests covering any new functionality
 - [ ] The documentation has been updated or is unnecessary
 - [ ] The changes have been verified
-- [ ] New dependencies are appropriate or there were no changes
+- [ ] New dependencies are appropriate or there were no changes

Makefile

@@ -14,7 +14,7 @@ install: ## Install script and dependencies
 	pipenv install --dev
 
 test: ## Run tests and print a coverage report
-	pipenv run coverage run --source=harvester -m pytest
+	pipenv run coverage run --source=harvester -m pytest -vv
 	pipenv run coverage report -m
 
 coveralls: test

Pipfile

@@ -5,14 +5,14 @@ verify_ssl = true
 
 [packages]
 click = "*"
+sentry-sdk = "*"
 sickle = "*"
 smart-open = {extras = ["s3"], version = "*"}
-sentry-sdk = "*"
 
 [dev-packages]
 bandit = "*"
-coverage = "*"
 black = "*"
+coverage = "*"
 flake8 = "*"
 isort = "*"
 mypy = "*"