Merge pull request #46 from davips/davipsbranch

Better Makefile to avoid problems with venv inside project folder.

CONTRIBUTING.md

@@ -48,11 +48,12 @@ following rules before submitting a pull request:
    [coding-guidelines](http://scikit-learn.org/dev/developers/contributing.html#coding-guidelines)
    as for scikit-learn.
 
--  When applicable, use the validation tools and other code in the
-   `pymfe._internal` submodule.
+-  When applicable, use the validation tools cited below (pytest, pylint and mypy).
+
+-  Avoid reinventing the wheel, reuse the functions from the `pymfe._internal` submodule.
 
 -  If your pull request addresses an issue, please use the title to describe
-   the issue and mention the issue number in the pull request description to
+   the issue and mention the issue number (using #) in the pull request description to
    ensure a link is created to the original issue.
 
 -  All public methods should have informative docstrings with sample
@@ -65,6 +66,10 @@ following rules before submitting a pull request:
    to indicate you are working on something to avoid duplicated work,
    request a broad review of functionality or API, or seek collaborators.
 
+-  Don't forget to activate your virtual environment, if any:
+
+        $ source venv/bin/activate
+
 -  All other tests pass when everything is rebuilt from scratch. On
    Unix-like systems, check with (from the top-level source folder):
 

Makefile

@@ -1,4 +1,4 @@
-all: clean code-check test-cov
+all: install-dev code-check test-cov
 
 clean:
 	find . -name "*.so" -o -name "*.pyc" -o -name "*.md5" -o -name "*.pyd" -o -name "*~" | xargs rm -f
@@ -25,8 +25,11 @@ pypi: clean
 	python3 setup.py sdist bdist_wheel
 	twine upload dist/*
 
-install-dev: install
+install-dev: install-dev-pymfe
 	pip install -U -r requirements-dev.txt
 
 install:
 	pip install -U -r requirements.txt
+
+install-dev-pymfe: install
+	pip install -e .