troughly pass all versions trough parsed versions

Author: RonnyPfannschmidt

CHANGELOG.rst

@@ -1,8 +1,9 @@
-v1.13.0
+v2.0.0
 ========
 
 * fix regression caused by the fix of #101
   * assert types for version dumping
+  * strictly pass all versions trough parsed version metadata
 
 v1.12.0
 =======

setuptools_scm/__init__.py

@@ -4,9 +4,10 @@
 """
 import os
 import sys
+import warnings
 
 from .utils import trace
-from .version import format_version
+from .version import format_version, meta, ScmVersion
 from .discover import find_matching_entrypoint
 
 PRETEND_KEY = 'SETUPTOOLS_SCM_PRETEND_VERSION'
@@ -60,10 +61,17 @@ def dump_version(root, version, write_to, template=None):
 def _do_parse(root, parse):
     pretended = os.environ.get(PRETEND_KEY)
     if pretended:
-        return pretended
+        return meta(pretended)
 
     if parse:
-        version = parse(root) or _version_from_entrypoint(
+        parse_result = parse(root)
+        if isinstance(parse_result, string_types):
+            warnings.warn(
+                "version parse result was a string\n"
+                "please return a parsed version",
+                category=DeprecationWarning)
+            parse_result = ScmVersion(parse_result)
+        version = parse_result or _version_from_entrypoint(
             root, 'setuptools_scm.parse_scm_fallback')
     else:
         # include fallbacks after dropping them from the main entrypoint
@@ -105,13 +113,10 @@ def get_version(root='.',
     parsed_version = _do_parse(root, parse)
 
     if parsed_version:
-        if isinstance(parsed_version, string_types):
-            version_string = parsed_version
-        else:
-            version_string = format_version(
-                parsed_version,
-                version_scheme=version_scheme,
-                local_scheme=local_scheme)
+        version_string = format_version(
+            parsed_version,
+            version_scheme=version_scheme,
+            local_scheme=local_scheme)
         dump_version(
             root=root,
             version=version_string,

setuptools_scm/hacks.py

@@ -1,5 +1,6 @@
 import os
 from .utils import data_from_mime, trace
+from .version import meta
 
 
 def parse_pkginfo(root):
@@ -9,4 +10,4 @@ def parse_pkginfo(root):
     data = data_from_mime(pkginfo)
     version = data.get('Version')
     if version != 'UNKNOWN':
-        return version
+        return meta(version)

testing/test_basic_api.py

@@ -74,7 +74,9 @@ def test_dump_version(tmpdir):
     ast.parse(content)
 
 
-def test_parse_plain():
+def test_parse_plain(recwarn):
     def parse(root):
         return 'tricked you'
     assert setuptools_scm.get_version(parse=parse) == 'tricked you'
+    assert str(recwarn.pop().message) == \
+        'version parse result was a string\nplease return a parsed version'