Raise NotImplementedError instead of returning it (#158)

* Raise NotImplementedError instead of returning it

Signed-off-by: Jono Yang <[email protected]>

* Update CHANGELOG.rst

Signed-off-by: Jono Yang <[email protected]>

---------

Signed-off-by: Jono Yang <[email protected]>

Author: JonoYang

CHANGELOG.rst

@@ -1,6 +1,11 @@
 Changelog
 =========
 
+Version v31.0.0
+----------------
+
+- Raise instead of returning ``NotImplementedError`` in ``version_range.VersionRange`` methods. https://github.com/aboutcode-org/univers/pull/158
+
 Version v30.12.1
 ----------------
 

src/univers/version_range.py

@@ -78,15 +78,15 @@ def from_native(cls, string):
         Return a VersionRange built from a scheme-specific, native version range
         ``string``. Subclasses can implement.
         """
-        return NotImplementedError
+        raise NotImplementedError
 
     @classmethod
     def from_natives(cls, strings):
         """
         Return a VersionRange built from a ``strings`` list of scheme-
         specific native version range strings. Subclasses can implement.
         """
-        return NotImplementedError
+        raise NotImplementedError
 
     def to_native(self, *args, **kwargs):
         """
@@ -95,7 +95,7 @@ def to_native(self, *args, **kwargs):
         extra arguments (such as a package name that some scheme may require
         like for deb and rpm.)
         """
-        return NotImplementedError
+        raise NotImplementedError
 
     @classmethod
     def from_string(cls, vers, simplify=False, validate=False):
@@ -176,7 +176,7 @@ def from_versions(cls, sequence):
         such as ["3.0.0", "1.0.1b", "3.0.2", "0.9.7a", "1.1.1ka"]
         """
         if not cls.scheme or not cls.version_class:
-            return NotImplementedError
+            raise NotImplementedError
 
         constraints = []
         for version in sequence: