Modernize to Python 3.11+ with UV and Ruff

This is a comprehensive modernization of python-gerritclient to modern
Python standards and tooling as of 2025.

## Major Changes

### Python Version Support
- **Drop support for Python 2.7, 3.5, 3.6**
- **Add support for Python 3.11, 3.12, 3.13**
- Bump version to 1.0.0 to signify breaking change

### Dependency Management & Tooling
- Add `pyproject.toml` with UV + Ruff configuration
- Replace pip/virtualenv workflow with UV (10-100x faster)
- Replace flake8/hacking with Ruff (100x faster linting)
- Replace black with Ruff format (30x faster, >99.9% compatible)
- Add comprehensive tool configuration in pyproject.toml

### Code Modernization
- Remove `six` library dependency (12 occurrences across 7 files)
- Replace `@six.add_metaclass(abc.ABCMeta)` with `class Foo(abc.ABC)`
- Replace deprecated `@abc.abstractproperty` with `@property @abstractmethod` (9 occurrences)
- Remove explicit `object` inheritance (unnecessary in Python 3)
- Fix `dict.values()` bug in base.py:86 - wrap with `list()` for Python 3 compatibility
- Replace `six.text_type()` with built-in `str()`

### Testing Infrastructure
- Replace standalone `mock` with `unittest.mock` (7 test files updated)
- Update tox.ini for Python 3.11-3.13 testing
- Add pytest and pytest-cov as test dependencies
- Add mypy for type checking support

### CI/CD
- Add GitHub Actions workflow (.github/workflows/test.yml)
- Test matrix for Python 3.11, 3.12, 3.13
- Automated linting, formatting checks, and test coverage
- Replace Travis CI configuration

### Documentation
- Update README.md with modern installation instructions
- Add UV installation and usage guide
- Document Python 3.11+ requirement
- Provide both UV and pip installation methods

### Code Quality
- All code formatted with Ruff (38 files reformatted)
- All code checked and auto-fixed with Ruff linter
- Removed deprecated Python 2/3 compatibility patterns
- Modernized string formatting (ruff auto-fixes applied)

## Files Modified
- 44 Python source files updated
- 3 new files created (pyproject.toml, .github/workflows/test.yml, uv.lock)
- 3 configuration files updated (setup.cfg, tox.ini, README.md)

## Testing
- ✅ `uv sync` completes successfully
- ✅ `uv run gerrit --help` works correctly
- ✅ All dependencies install cleanly
- ✅ Code is properly formatted and linted

## Breaking Changes
- **Python 2.7 no longer supported**
- **Python 3.5, 3.6 no longer supported**
- **Minimum Python version is now 3.11**
- six dependency removed (breaking for any code importing six)

This modernization sets up python-gerritclient for long-term maintenance
and positions it as a modern, well-maintained Python 3.11+ project.

Author: claude

.github/workflows/test.yml

@@ -0,0 +1,44 @@
+name: Test
+
+on:
+  push:
+    branches: [main, claude/**]
+  pull_request:
+    branches: [main]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.11", "3.12", "3.13"]
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v5
+        with:
+          version: "latest"
+
+      - name: Set up Python ${{ matrix.python-version }}
+        run: uv python install ${{ matrix.python-version }}
+
+      - name: Install dependencies
+        run: uv sync --all-extras
+
+      - name: Lint with ruff
+        run: uv run ruff check .
+
+      - name: Format check with ruff
+        run: uv run ruff format --check .
+
+      - name: Run tests
+        run: uv run pytest --cov=gerritclient --cov-report=term-missing --cov-report=xml
+
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v4
+        if: matrix.python-version == '3.11'
+        with:
+          file: ./coverage.xml
+          fail_ci_if_error: false

README.md

@@ -5,32 +5,84 @@
 # python-gerritclient
 CLI tool and Python API wrapper for Gerrit Code Review
 
+## Requirements
+
+**Python 3.11+** is required. This project uses modern Python features and tooling.
+
 ## Quick Start
 
-### Command Line Tool
-1. Clone `python-gerritclient` repository: `git clone https://github.com/tivaliy/python-gerritclient.git`.
-2. Configure `settings.yaml` file (in `gerritclient/settings.yaml`) to meet your requirements.
+### Command Line Tool (Recommended: Using UV)
+
+[UV](https://docs.astral.sh/uv/) is a fast, modern Python package manager. Recommended for the best experience.
+
+1. Install UV (if not already installed):
+   ```bash
+   curl -LsSf https://astral.sh/uv/install.sh | sh
+   ```
 
+2. Clone the repository:
+   ```bash
+   git clone https://github.com/tivaliy/python-gerritclient.git
+   cd python-gerritclient
+   ```
+
+3. Configure `settings.yaml` file (in `gerritclient/settings.yaml`):
     ```yaml
-       url: http://review.example.com
-       auth_type: basic
-       username: admin
-       password: "1234567890aaWmmflSl+ZlOPs23Dffn"
+    url: http://review.example.com
+    auth_type: basic
+    username: admin
+    password: "1234567890aaWmmflSl+ZlOPs23Dffn"
     ```
 
-    * `url` can be specified according to the following format `<scheme>://<host>:<port>`, e.g. `https://review.openstack.org`
-    * `auth_type` specifies HTTP authentication scheme (`basic` or `digest`), can be omitted, then all requests will be anonymous with respective restrictions
-    * `username` and `password` - user credentials from Gerrit system (Settings &#8594; HTTP Password)
+    * `url` - Gerrit server URL in format `<scheme>://<host>:<port>` (e.g., `https://review.openstack.org`)
+    * `auth_type` - HTTP authentication scheme (`basic` or `digest`), omit for anonymous access
+    * `username` and `password` - user credentials from Gerrit (Settings → HTTP Password)
+
+4. Install dependencies and run:
+   ```bash
+   uv sync
+   uv run gerrit --help
+   ```
+
+5. Run commands:
+   ```bash
+   uv run gerrit plugin list
+   uv run gerrit account list "john"
+   ```
 
-3. Create isolated Python environment `virtualenv gerritclient_venv` and activate it `source gerritclient_venv/bin/activate`.
-4. Install `python-gerritclient` with all necessary dependencies: `pip install python-gerritclient/.`.
-5. (Optional) Add gerrit command bash completion `gerrit complete | sudo tee /etc/bash_completion.d/gc.bash_completion > /dev/null`
-6. Run `gerrit` command with required options, e.g. `gerrit plugin list`. To see all available commands run `gerrit --help`.
+### Command Line Tool (Alternative: Using pip)
 
-### Library
-1. Clone `python-gerritclient` repository: `git clone https://github.com/tivaliy/python-gerritclient.git`.
-2. Create isolated Python environment `virtualenv gerritclient_venv` and activate it `source gerritclient_venv/bin/activate`.
-3. Install `python-gerritclient` with all necessary dependencies: `pip install python-gerritclient/.`.
+1. Clone the repository:
+   ```bash
+   git clone https://github.com/tivaliy/python-gerritclient.git
+   cd python-gerritclient
+   ```
+
+2. Configure `settings.yaml` (same as above)
+
+3. Install with pip:
+   ```bash
+   python3 -m venv venv
+   source venv/bin/activate  # On Windows: venv\Scripts\activate
+   pip install -e .
+   ```
+
+4. Run commands:
+   ```bash
+   gerrit --help
+   gerrit plugin list
+   ```
+
+### Library Usage
+
+Install the package:
+```bash
+# With UV
+uv add python-gerritclient
+
+# With pip
+pip install python-gerritclient
+```
 
 ```python
 from gerritclient import client

gerritclient/client.py

@@ -15,17 +15,16 @@
 
 import json
 import os
-import requests
-
-import gerritclient
 
+import requests
 from requests import auth
 
-from gerritclient.common import utils
+import gerritclient
 from gerritclient import error
+from gerritclient.common import utils
 
 
-class APIClient(object):
+class APIClient:
     """This class handles API requests."""
 
     def __init__(self, url, auth_type=None, username=None, password=None):
@@ -49,11 +48,10 @@ def __init__(self, url, auth_type=None, username=None, password=None):
         self._auth = None
         if auth_type:
             if not all((self._username, self._password)):
-                raise ValueError('Username and password must be specified.')
-            auth_types = {'basic': auth.HTTPBasicAuth,
-                          'digest': auth.HTTPDigestAuth}
+                raise ValueError("Username and password must be specified.")
+            auth_types = {"basic": auth.HTTPBasicAuth, "digest": auth.HTTPDigestAuth}
             if auth_type not in auth_types:
-                raise ValueError('Unsupported auth_type {}'.format(auth_type))
+                raise ValueError(f"Unsupported auth_type {auth_type}")
             self._auth = auth_types[auth_type](self._username, self._password)
 
         if self.is_authed:
@@ -71,8 +69,7 @@ def is_authed(self):
     def _make_common_headers():
         """Returns a dict of HTTP headers common for all requests."""
 
-        return {'Content-Type': 'application/json',
-                'Accept': 'application/json'}
+        return {"Content-Type": "application/json", "Accept": "application/json"}
 
     def _make_session(self):
         """Initializes a HTTP session."""
@@ -135,8 +132,7 @@ def get_request(self, api, params=None):
         self._raise_for_status_with_info(resp)
         return self._decode_content(resp)
 
-    def post_request_raw(self, api, data=None, json_data=None,
-                         content_type=None):
+    def post_request_raw(self, api, data=None, json_data=None, content_type=None):
         """Make a POST request to specific API and return raw response.
 
         :param api: API endpoint (path)
@@ -150,12 +146,11 @@ def post_request_raw(self, api, data=None, json_data=None,
         # Some POST requests require 'Content-Type' value other
         # than default 'application/json'
         if content_type is not None:
-            self.session.headers.update({'Content-Type': content_type})
+            self.session.headers.update({"Content-Type": content_type})
 
         return self.session.post(url, data=data, json=json_data)
 
-    def post_request(self, api, data=None, json_data=None,
-                     content_type=None):
+    def post_request(self, api, data=None, json_data=None, content_type=None):
         """Make POST request to specific API with some data."""
 
         resp = self.post_request_raw(api, data, json_data, content_type)
@@ -175,7 +170,7 @@ def _decode_content(response):
             return {}
 
         # Some responses can be of 'text/plain' Content-Type
-        if 'text/plain' in response.headers.get('Content-Type'):
+        if "text/plain" in response.headers.get("Content-Type"):
             return response.text
 
         # Remove ")]}'" prefix from response, that is used to prevent XSSI
@@ -194,9 +189,10 @@ def get_settings(file_path=None):
 
     config = None
 
-    user_config = os.path.join(os.path.expanduser('~'), '.config',
-                               'gerritclient', 'settings.yaml')
-    local_config = os.path.join(os.path.dirname(__file__), 'settings.yaml')
+    user_config = os.path.join(
+        os.path.expanduser("~"), ".config", "gerritclient", "settings.yaml"
+    )
+    local_config = os.path.join(os.path.dirname(__file__), "settings.yaml")
 
     if file_path is not None:
         config = file_path
@@ -211,22 +207,19 @@ def get_settings(file_path=None):
 
     try:
         config_data = utils.read_from_file(config)
-    except (OSError, IOError):
-        msg = "Could not read settings from {0}".format(file_path)
+    except OSError:
+        msg = f"Could not read settings from {file_path}"
         raise error.InvalidFileException(msg)
     return config_data
 
 
 def connect(url, auth_type=None, username=None, password=None):
     """Creates API connection."""
 
-    return APIClient(url,
-                     auth_type=auth_type,
-                     username=username,
-                     password=password)
+    return APIClient(url, auth_type=auth_type, username=username, password=password)
 
 
-def get_client(resource, version='v1', connection=None):
+def get_client(resource, version="v1", connection=None):
     """Gets an API client for a resource
 
     python-gerritclient provides access to Gerrit Code Review's API
@@ -245,13 +238,13 @@ def get_client(resource, version='v1', connection=None):
     """
 
     version_map = {
-        'v1': {
-            'account': gerritclient.v1.account,
-            'change': gerritclient.v1.change,
-            'group': gerritclient.v1.group,
-            'plugin': gerritclient.v1.plugin,
-            'project': gerritclient.v1.project,
-            'server': gerritclient.v1.server
+        "v1": {
+            "account": gerritclient.v1.account,
+            "change": gerritclient.v1.change,
+            "group": gerritclient.v1.group,
+            "plugin": gerritclient.v1.plugin,
+            "project": gerritclient.v1.project,
+            "server": gerritclient.v1.server,
         }
     }