search

collective / collective.transmogrifier

A configurable pipeline, aimed at transforming content for import and export
Other
5 stars 8 forks source link

Prepare a Python 3 release of the package #30

Closed ericof closed 2 years ago

ericof commented 2 years ago

The current version (2.0.0) is not installable using pip due to an issue with python_requires, instead of fixing just that we should prepare a Python 3 only version of the package.

Fixes #29

ericof commented 2 years ago

@wesleybl Any opinions about this?

wesleybl commented 2 years ago

This must be fixed:

https://github.com/collective/collective.transmogrifier/blob/c8f740bc2c91be1d26da96ba5ccf85b55d9ead1b/src/collective/transmogrifier/transmogrifier.py#L15-L19

wesleybl commented 2 years ago

This must be removed:

https://github.com/collective/collective.transmogrifier/blob/c8f740bc2c91be1d26da96ba5ccf85b55d9ead1b/src/collective/transmogrifier/sections/splitter.py#L65

wesleybl commented 2 years ago

You can change the Archetype reference to dexterity:

https://github.com/collective/collective.transmogrifier/blob/d3f49b3647f2fa4a66c1b9e94a90592c563c76b8/docs/source/transmogrifier.rst#keys-are-fields

ericof commented 2 years ago

@wesleybl, @mauritsvanrees: I will merge and make a release (as this is a blocker for us at the moment), but I want to go back to discussing tox and the approach of virtualenvs for 5.2 and 6.0