Copilot commented on code in PR #1284:
URL: 
https://github.com/apache/cassandra-python-driver/pull/1284#discussion_r3025858522


##########
docs/upgrading.rst:
##########
@@ -4,9 +4,51 @@ Upgrading
 .. toctree::
    :maxdepth: 1
 
+Upgrading to 3.30.0
+-------------------
+Version 3.30.0 of the Python driver is the first release since the driver's 
donation
+to the Apache Software Foundation (ASF).
+
+Supported Python Versions
+^^^^^^^^^^^^^^^^^^^^^^^^^
+An individual version of the Python driver aims to officially support all 
Python runtimes
+that are not end-of-life (EOL) at the time of that versions release.  For 
3.30.0 this policy
+entails support for Python 3.10 through Python 3.14.  The driver will likely 
continue to work
+reasonably well on older Python runtimes but only these versions are 
officially supported.
+
+Conversion to pyproject.toml
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+With this release we are moving away from the install and configuration 
process based on
+setup.py and towards the use of pyproject.toml.  As a result of this change 
(and in keeping
+with the uses of pyproject.toml) configuration of a driver build is now 
declarative.  All
+build options should be specified in pyproject.toml and overrides via 
command-line flags or
+environment varaibles are no longer supported.  Please consult
+`CASSPYTHON-7 <https://issues.apache.org/jira/browse/CASSPYTHON-7>`_ for 
additional details.
+
+Event Loop Deprecation
+^^^^^^^^^^^^^^^^^^^^^^
+With this release the eventlet, gevent and Twisted event loops are considered 
deprecated.  Use
+of these event loops in this version will generate a warning to this effect.  
We are planning on
+removing these event loops in their entirety in the next minor release.  
Please consult
+`CASSPYTHON-12 <https://issues.apache.org/jira/browse/CASSPYTHON-12>`_ for 
additional details.
+
+Removal of Win32 Wheels
+^^^^^^^^^^^^^^^^^^^^^^^
+As of this release we will no longer be offering wheels for Win32 platforms.  
Wheels for other
+Windows platforms will continue to be deployed to PyPi.  Please consult

Review Comment:
   Spelling/branding: PyPI is typically capitalized "PyPI" (not "PyPi").
   ```suggestion
   Windows platforms will continue to be deployed to PyPI.  Please consult
   ```



##########
docs/upgrading.rst:
##########
@@ -4,9 +4,51 @@ Upgrading
 .. toctree::
    :maxdepth: 1
 
+Upgrading to 3.30.0
+-------------------
+Version 3.30.0 of the Python driver is the first release since the driver's 
donation
+to the Apache Software Foundation (ASF).
+
+Supported Python Versions
+^^^^^^^^^^^^^^^^^^^^^^^^^
+An individual version of the Python driver aims to officially support all 
Python runtimes
+that are not end-of-life (EOL) at the time of that versions release.  For 
3.30.0 this policy

Review Comment:
   Typo/grammar: "that versions release" is missing the possessive apostrophe; 
it should be "that version's release".
   ```suggestion
   that are not end-of-life (EOL) at the time of that version's release.  For 
3.30.0 this policy
   ```



##########
docs/upgrading.rst:
##########
@@ -4,9 +4,51 @@ Upgrading
 .. toctree::
    :maxdepth: 1
 
+Upgrading to 3.30.0
+-------------------
+Version 3.30.0 of the Python driver is the first release since the driver's 
donation
+to the Apache Software Foundation (ASF).
+
+Supported Python Versions
+^^^^^^^^^^^^^^^^^^^^^^^^^
+An individual version of the Python driver aims to officially support all 
Python runtimes
+that are not end-of-life (EOL) at the time of that versions release.  For 
3.30.0 this policy
+entails support for Python 3.10 through Python 3.14.  The driver will likely 
continue to work
+reasonably well on older Python runtimes but only these versions are 
officially supported.
+
+Conversion to pyproject.toml
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+With this release we are moving away from the install and configuration 
process based on
+setup.py and towards the use of pyproject.toml.  As a result of this change 
(and in keeping
+with the uses of pyproject.toml) configuration of a driver build is now 
declarative.  All

Review Comment:
   Grammar: "uses of pyproject.toml" should be "use of pyproject.toml".
   ```suggestion
   with the use of pyproject.toml) configuration of a driver build is now 
declarative.  All
   ```



##########
docs/upgrading.rst:
##########
@@ -4,9 +4,51 @@ Upgrading
 .. toctree::
    :maxdepth: 1
 
+Upgrading to 3.30.0
+-------------------
+Version 3.30.0 of the Python driver is the first release since the driver's 
donation
+to the Apache Software Foundation (ASF).
+
+Supported Python Versions
+^^^^^^^^^^^^^^^^^^^^^^^^^
+An individual version of the Python driver aims to officially support all 
Python runtimes
+that are not end-of-life (EOL) at the time of that versions release.  For 
3.30.0 this policy
+entails support for Python 3.10 through Python 3.14.  The driver will likely 
continue to work
+reasonably well on older Python runtimes but only these versions are 
officially supported.
+
+Conversion to pyproject.toml
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+With this release we are moving away from the install and configuration 
process based on
+setup.py and towards the use of pyproject.toml.  As a result of this change 
(and in keeping
+with the uses of pyproject.toml) configuration of a driver build is now 
declarative.  All
+build options should be specified in pyproject.toml and overrides via 
command-line flags or
+environment varaibles are no longer supported.  Please consult

Review Comment:
   Typo: "environment varaibles" is misspelled; should be "environment 
variables".
   ```suggestion
   environment variables are no longer supported.  Please consult
   ```



##########
cassandra/connection.py:
##########
@@ -107,7 +107,7 @@ def decompress(byts):
         return snappy.decompress(byts)
     locally_supported_compressions['snappy'] = (snappy.compress, decompress)
 
-DRIVER_NAME, DRIVER_VERSION = 'DataStax Python Driver', 
sys.modules['cassandra'].__version__
+DRIVER_NAME, DRIVER_VERSION = 'Apache Cassandra Python Driver', 
sys.modules['cassandra'].__version__

Review Comment:
   After changing STARTUP's DRIVER_NAME, the DataStax Insights payload still 
hard-codes the old name (cassandra/datastax/insights/reporter.py:170 uses 
"DataStax Python Driver"). If Insights is expected to reflect the same driver 
identity change, this should be updated to match (or documented/justified if it 
must remain DataStax-branded).
   ```suggestion
   DRIVER_NAME = 'Apache Cassandra Python Driver'
   cassandra_module = sys.modules.get('cassandra')
   if cassandra_module is not None and hasattr(cassandra_module, '__version__'):
       DRIVER_VERSION = cassandra_module.__version__
   else:
       DRIVER_VERSION = 'unknown'
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to