Merge drop_py2

.github/workflows/autofmt.yml

@@ -0,0 +1,37 @@
+on:
+  pull_request:
+    branches: [drop_py2]  # TODO: add `master`
+
+jobs:
+    formatter:
+        name: auto-formatter
+        runs-on: windows-latest
+        steps:
+          - name: Checkout
+            uses: actions/checkout@v3
+          - name: Set up Python
+            uses: actions/setup-python@v4
+            with:
+              python-version: 3.7
+          - name: Install black
+            run: pip install black==22.12.0
+          # PRs from the forked repo will trigger format-checking only.
+          # Auto-commit to the branch from a forked repo will fail without
+          # any access tokens or permissions.
+          # So formatting and auto-commit will be triggered by PRs from the
+          # base repo only.
+          - if: github.repository != github.event.pull_request.head.repo.full_name
+            name: Check style
+            run: python -m black comtypes/. --check --diff --color
+          - if: github.repository == github.event.pull_request.head.repo.full_name
+            name: Format
+            run: python -m black comtypes/.
+          - if: github.repository == github.event.pull_request.head.repo.full_name
+            name: Auto-commit
+            uses: stefanzweifel/git-auto-commit-action@v4
+            with:
+              branch: ${{ github.head_ref }}
+              commit_message: apply automatic formatter
+              commit_user_name: github-actions[bot]
+              commit_user_email: 41898282+github-actions[bot]@users.noreply.github.com
+              commit_author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

CONTRIBUTING.md

@@ -9,7 +9,6 @@ We appreciate all contributions, from reporting bugs to implementing new feature
 ## Table of contents
 - [To keep the community healthy and sustainable :busts_in_silhouette:](#to-keep-the-community-healthy-and-sustainable-busts_in_silhouette)
 - [Reporting bugs :bug:](#reporting-bugs-bug)
-- [Breaking troubles down :flashlight:](#breaking-troubles-down-flashlight)
 - [Suggesting enhancements :sparkles:](#suggesting-enhancements-sparkles)
 - [Contributing to the codebase :open_file_folder:](#contributing-to-the-codebase-open_file_folder)
 - [Contributing to documentation :books:](#contributing-to-documentation-books)
@@ -19,19 +18,19 @@ We appreciate all contributions, from reporting bugs to implementing new feature
 
 If you have to understand all that this package has to offer, it would require an enormous amount of knowledge.
 
-- `Python`:snake:
-- `C`-lang:computer:
-- COM interface, implementation, client and server:door:
-- COM type library functionalities:wrench:
+- `Python` :snake:
+- `C`-lang :computer:
+- COM interface, implementation, client and server :door:
+- COM type library functionalities :wrench:
 
 However, there is **no means to say** that you must understand all of these things to be a contributor.
 
 The purpose of this document is to provide a pathway for you to contribute to this community even if you only know a small portion of this package.
 
 Please keep the followings in your mind:
-- :point_right:Please follow the following guidelines when posting a [GitHub Pull Request](https://github.com/enthought/comtypes/pulls) or filing a [GitHub Issue](https://github.com/enthought/comtypes/issues) on this project.
-- :bow:The community may not be able to process and reply to your issue or PR right-away. Participants in the community have a lot of work to do besides `comtypes`, but they would try their best.
-- :book:For code of conduct, please read [Contributor Covenant](https://www.contributor-covenant.org/).
+- :point_right: Please follow the following guidelines when posting a [GitHub Pull Request](https://github.com/enthought/comtypes/pulls) or filing a [GitHub Issue](https://github.com/enthought/comtypes/issues) on this project.
+- :bow: The community may not be able to process and reply to your issue or PR right-away. Participants in the community have a lot of work to do besides `comtypes`, but they would try their best.
+- :book: For code of conduct, please read [Contributor Covenant](https://www.contributor-covenant.org/).
 
 ## Reporting bugs :bug:
 
@@ -91,7 +90,7 @@ Those `.py` files act like ”caches”.
 
 If there are some problems with the developing code base, partial or non-executable modules might be created in `.../comtypes/gen/...`.  
 Importing them will cause some error.  
-If that happens, you should run `python -m clear_comtypes_cache` to clear those caches.  
+If that happens, you should run `python -m comtypes.clear_cache` to clear those caches.  
 The command will delete the entire `.../comtypes/gen` directory.  
 Importing `comtypes.client` will restore the directory and `__init__.py` file.
 

appveyor.yml

@@ -9,8 +9,6 @@ shallow_clone: true
 
 environment:
    matrix:
-     - py: Python27
-     - py: Python27-x64
      - py: Python37
      - py: Python37-x64
      - py: Python38

comtypes/GUID.py

@@ -1,19 +1,10 @@
 from ctypes import *
 import sys
 
-if sys.version_info >= (2, 6):
-    def binary(obj):
-        return bytes(obj)
-else:
-    def binary(obj):
-        return buffer(obj)
-
-if sys.version_info >= (3, 0):
-    text_type = str
-    base_text_type = str
-else:
-    text_type = unicode
-    base_text_type = basestring
+
+def binary(obj):
+    return bytes(obj)
+
 
 BYTE = c_byte
 WORD = c_ushort
@@ -31,25 +22,24 @@ def binary(obj):
 # Note: Comparing GUID instances by comparing their buffers
 # is slightly faster than using ole32.IsEqualGUID.
 
+
 class GUID(Structure):
-    _fields_ = [("Data1", DWORD),
-                ("Data2", WORD),
-                ("Data3", WORD),
-                ("Data4", BYTE * 8)]
+    _fields_ = [("Data1", DWORD), ("Data2", WORD), ("Data3", WORD), ("Data4", BYTE * 8)]
 
     def __init__(self, name=None):
         if name is not None:
-            _CLSIDFromString(text_type(name), byref(self))
+            _CLSIDFromString(str(name), byref(self))
 
     def __repr__(self):
-        return 'GUID("%s")' % text_type(self)
+        return 'GUID("%s")' % str(self)
 
     def __unicode__(self):
         p = c_wchar_p()
         _StringFromCLSID(byref(self), byref(p))
         result = p.value
         _CoTaskMemFree(p)
         return result
+
     __str__ = __unicode__
 
     def __cmp__(self, other):
@@ -61,29 +51,27 @@ def __bool__(self):
         return self != GUID_null
 
     def __eq__(self, other):
-        return isinstance(other, GUID) and \
-               binary(self) == binary(other)
+        return isinstance(other, GUID) and binary(self) == binary(other)
 
     def __hash__(self):
         # We make GUID instances hashable, although they are mutable.
         return hash(binary(self))
 
     def copy(self):
-        return GUID(text_type(self))
+        return GUID(str(self))
 
     @classmethod
     def from_progid(cls, progid):
-        """Get guid from progid, ...
-        """
+        """Get guid from progid, ..."""
         if hasattr(progid, "_reg_clsid_"):
             progid = progid._reg_clsid_
         if isinstance(progid, cls):
             return progid
-        elif isinstance(progid, base_text_type):
+        elif isinstance(progid, str):
             if progid.startswith("{"):
                 return cls(progid)
             inst = cls()
-            _CLSIDFromProgID(text_type(progid), byref(inst))
+            _CLSIDFromProgID(str(progid), byref(inst))
             return inst
         else:
             raise TypeError("Cannot construct guid from %r" % progid)