move project metadata to pyproject.toml (#555)

also: replace flake8 by ruff.

Author: ThomasWaldmann

docs/conf.py

@@ -11,11 +11,11 @@
 # All configuration values have a default; values that are commented out
 # serve to show the default.
 
-import sys, os
-
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
 # documentation root, use os.path.abspath to make it absolute, like shown here.
+#import os
+#import sys
 #sys.path.insert(0, os.path.abspath('..'))
 
 # -- General configuration -----------------------------------------------------

msgpack/__init__.py

@@ -2,7 +2,6 @@
 from .ext import ExtType, Timestamp
 
 import os
-import sys
 
 
 version = (1, 0, 5)

msgpack/ext.py

@@ -1,6 +1,5 @@
 from collections import namedtuple
 import datetime
-import sys
 import struct
 
 
@@ -20,8 +19,9 @@ def __new__(cls, code, data):
 class Timestamp:
     """Timestamp represents the Timestamp extension type in msgpack.
 
-    When built with Cython, msgpack uses C methods to pack and unpack `Timestamp`. When using pure-Python
-    msgpack, :func:`to_bytes` and :func:`from_bytes` are used to pack and unpack `Timestamp`.
+    When built with Cython, msgpack uses C methods to pack and unpack `Timestamp`.
+    When using pure-Python msgpack, :func:`to_bytes` and :func:`from_bytes` are used to pack and
+    unpack `Timestamp`.
 
     This class is immutable: Do not override seconds and nanoseconds.
     """
@@ -39,7 +39,7 @@ def __init__(self, seconds, nanoseconds=0):
             Number of nanoseconds to add to `seconds` to get fractional time.
             Maximum is 999_999_999.  Default is 0.
 
-        Note: Negative times (before the UNIX epoch) are represented as negative seconds + positive ns.
+        Note: Negative times (before the UNIX epoch) are represented as neg. seconds + pos. ns.
         """
         if not isinstance(seconds, int):
             raise TypeError("seconds must be an integer")

msgpack/fallback.py

@@ -530,7 +530,7 @@ def _unpack(self, execute=EX_CONSTRUCT):
                     key = self._unpack(EX_CONSTRUCT)
                     if self._strict_map_key and type(key) not in (str, bytes):
                         raise ValueError("%s is not allowed for map key" % str(type(key)))
-                    if type(key) is str:
+                    if isinstance(key, str):
                         key = sys.intern(key)
                     ret[key] = self._unpack(EX_CONSTRUCT)
                 if self._object_hook is not None:

pyproject.toml

@@ -7,7 +7,52 @@ requires = [
 ]
 build-backend = "setuptools.build_meta"
 
+[project]
+name = "msgpack"
+dynamic = ["version"]
+license = {text="Apache 2.0"}
+authors = [{name="Inada Naoki", email="songofacandy@gmail.com"}]
+description = "MessagePack serializer"
+readme = "README.md"
+#keywords = ["python", "msgpack", "messagepack", "serializer", "serialization", "binary"]
+#requires-python = ">=3.8"
+classifiers = [
+#    "Development Status :: 5 - Production/Stable",
+#    "Operating System :: OS Independent",
+#    "Programming Language :: Python",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.8",
+    "Programming Language :: Python :: 3.9",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: Implementation :: CPython",
+    "Programming Language :: Python :: Implementation :: PyPy",
+    "Intended Audience :: Developers",
+    "License :: OSI Approved :: Apache Software License",
+]
+
+[project.urls]
+Homepage = "https://msgpack.org/"
+Documentation = "https://msgpack-python.readthedocs.io/"
+Repository = "https://github.com/msgpack/msgpack-python/"
+Tracker = "https://github.com/msgpack/msgpack-python/issues"
+#Changelog = "https://github.com/msgpack/msgpack-python/blob/main/ChangeLog.rst"
+
+[tool.setuptools.dynamic]
+version = {attr = "msgpack.__version__"}
+
 [tool.black]
 line-length = 100
 target-version = ["py37"]
 skip_string_normalization = true
+
+[tool.ruff]
+line-length = 100
+target-version = "py38"
+ignore = []
+
+[tool.ruff.per-file-ignores]
+"msgpack/__init__.py" = ["F401", "F403"]
+"msgpack/fallback.py" = ["E731"]
+"test/test_seq.py" = ["E501"]

setup.cfg

@@ -1,8 +1,6 @@
 #!/usr/bin/env python
-import io
 import os
 import sys
-from glob import glob
 from setuptools import setup, Extension
 from setuptools.command.build_ext import build_ext
 from setuptools.command.sdist import sdist

setup.py

@@ -1,7 +1,5 @@
 #!/usr/bin/env python
 
-import sys
-import pytest
 from msgpack import packb, unpackb
 
 

test/test_buffer.py

@@ -1,9 +1,7 @@
 #!/usr/bin/env python
 
-import pytest
 from array import array
 from msgpack import packb, unpackb
-import sys
 
 
 def make_array(f, data):

test/test_memoryview.py

@@ -33,7 +33,7 @@ def test_decode_pairs_hook():
     prod_sum = 1 * 2 + 3 * 4
     unpacked = unpackb(
         packed,
-        object_pairs_hook=lambda l: sum(k * v for k, v in l),
+        object_pairs_hook=lambda lst: sum(k * v for k, v in lst),
         use_list=1,
         strict_map_key=False,
     )
@@ -48,7 +48,7 @@ def test_only_one_obj_hook():
 def test_bad_hook():
     with raises(TypeError):
         packed = packb([3, 1 + 2j], default=lambda o: o)
-        unpacked = unpackb(packed, use_list=1)
+        unpackb(packed, use_list=1)
 
 
 def _arr_to_str(arr):