Merge pull request #161 from superannotateai/re-design-sdk

Re design sdk

Author: VaghinakDev

.coveragerc

@@ -0,0 +1,20 @@
+name: Build
+on:
+  push:
+    branches:
+      - re-design-sdk
+  pull_request:
+    types: [opened, synchronize, reopened]
+jobs:
+  sonarcloud:
+    name: SonarCloud
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+        with:
+          fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
+      - name: SonarCloud Scan
+        uses: SonarSource/sonarcloud-github-action@master
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

.dockerignore

@@ -2,6 +2,9 @@
 __pycache__/
 *.py[cod]
 *$py.class
+*mypy*
+*.egg*
+*DS_Store*
 
 # C extensions
 *.so
@@ -14,7 +17,6 @@ dist/
 downloads/
 eggs/
 .eggs/
-lib/
 lib64/
 parts/
 sdist/
@@ -54,7 +56,7 @@ coverage.xml
 # Translations
 *.mo
 *.pot
-
+.env
 # Django stuff:
 *.log
 local_settings.py
@@ -141,6 +143,7 @@ _mask.c
 tags
 temp
 debug_*
-
-# IDE
-.idea
+*.idea
+*.DS_Store
+htmlcov
+htmlcov

.github/workflows/build.yml

@@ -0,0 +1,44 @@
+repos:
+  - repo: 'https://gitlab.com/pycqa/flake8'
+    rev: 3.8.2
+    hooks:
+      - id: flake8
+        exclude:  src/lib/app/analytics | src/lib/app/converters | src/lib/app/input_convertors
+        name: Style Guide Enforcement (flake8)
+        args:
+          - '--max-line-length=120'
+          - --ignore=D100,D203,D405,W503,E203,E501,F841,E126,E712
+  - repo: 'https://github.com/asottile/pyupgrade'
+    rev: v2.4.3
+    hooks:
+      - id: pyupgrade
+        exclude: src/lib/app/analytics | src/lib/app/converters | src/lib/app/input_convertors
+        name: Upgrade syntax for newer versions of the language (pyupgrade)
+        args:
+          - '--py37-plus'
+  - repo: 'https://github.com/asottile/reorder_python_imports'
+    rev: v2.3.0
+    hooks:
+      - id: reorder-python-imports
+        exclude: src/lib/app/analytics | src/lib/app/converters | src/lib/app/input_convertors
+        name: 'Reorder Python imports (src, tests)'
+        args:
+          - '--application-directories'
+          - app
+  - repo: 'https://github.com/pre-commit/pre-commit-hooks'
+    rev: v3.1.0
+    hooks:
+      - id: check-byte-order-marker
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+  - repo: 'https://github.com/python/black'
+    rev: 19.10b0
+    hooks:
+      - id: black
+        name: Uncompromising Code Formatter (black)
+#  - repo: 'https://github.com/asottile/dead'
+#    rev: v1.3.0
+#    hooks:
+#      - id: dead
+files: src/
+exclude: src/lib/app/analytics | src/lib/app/converters | src/lib/app/input_convertors

.gitignore

@@ -1,66 +1,26 @@
-.PHONY: clean tests stress-tests test_coverage install lint docs dist check_formatting
+.PHONY: help coverage linter install mypy test validate
 
-MAKEFLAGS += -j1
+help:
+	@echo "Please use \`make <target>' where <target> is one of"
+	@echo "  coverage   to make source code coverage check"
+	@echo "  help       to show this help"
+	@echo "  test       to make tests running"
+	@echo "  validate   to make source code validation"
 
-PYTHON=python3
-PIP=pip3
-PYLINT=pylint
-PYTESTS=pytest
-COVERAGE=coverage
-
-tests: check_formatting docs
-	$(PYTESTS) -n auto --full-trace tests
-
-stress-tests: SA_STRESS_TESTS=1
-stress-tests: tests
-
-clean:
-	rm -rf superannotate.egg-info
-	rm -rf build
-	rm -rf dist
-	rm -rf htmlcov
-
-test_coverage: check_formatting
-	-$(PYTESTS) --cov=superannotate -n auto tests
-	$(COVERAGE) html
-	@echo "\033[95m\n\nCoverage successful! View the output at file://htmlcov/index.html.\n\033[0m"
+coverage:
+	tox -e coverage
 
 install:
-	$(PIP) install -e .
-
-lint: check_formatting
-	-$(PYLINT) --output-format=json superannotate/ | pylint-json2html -o pylint.html
-
-lint_tests:
-	-$(PYLINT) tests/*
-
-docs:
-	cd docs && make html SPHINXOPTS="-W"
-	@echo "\033[95m\n\nBuild successful! View the docs homepage at file://docs/build/html/index.html.\n\033[0m"
-
-dist:
-	-rm -rf dist/*
-	$(PYTHON) setup.py sdist
-	twine upload dist/*
-
-check_formatting:
-	yapf -p -r --diff superannotate
+	@echo "This function not implemented yet"
 
-docker_run_dev_env: docker_pull_dev_env
-	docker run -it -p 8888:8888 \
-		-v ${HOME}/.superannotate:/root/.superannotate \
-		-v $(pwd):/root/superannotate-python-sdk \
-		superannotate/pythonsdk-dev-env
+linter:
+	tox -e linter
 
-docker_run_dev_env_local_copy: docker_build_dev_env_from_local_copy
-	docker run -it -p 8888:8888 \
-	    -v ${HOME}/.superannotate:/root/.superannotate \
-	    -v $(pwd):/root/superannotate-python-sdk \
-	    superannotate/pythonsdk-dev-env \
-	    bash -c "pip install -e superannotate-python-sdk && jupyter lab --allow-root --NotebookApp.token='' --NotebookApp.password='' --no-browser --ip 0.0.0.0"
+mypy:
+	tox -e mypy
 
-docker_build_dev_env_from_local_copy:
-	docker build -t superannotate/pythonsdk-dev-env:latest -f Dockerfile_dev_env .
+test:
+	tox
 
-docker_pull_dev_env:
-	docker pull superannotate/pythonsdk-dev-env:latest
+validate:
+	tox -e pre-commit