search

yadage / yadage-schemas

schemas for packtivity and yadage for use in cap
MIT License
2 stars 1 forks source link

Specify `six` dependency #35

Closed matthewfeickert closed 2 years ago

matthewfeickert commented 3 years ago

Coming here from https://github.com/recast-hep/recast-atlas/issues/76, yadageschemas uses six in

https://github.com/yadage/yadage-schemas/blob/260fbc40ec5b8afa7d59c0dd00591e02e13774e4/yadageschemas/dialects/raw_with_defaults.py#L118

but doesn't specify this in its requirements

https://github.com/yadage/yadage-schemas/blob/260fbc40ec5b8afa7d59c0dd00591e02e13774e4/setup.py#L11-L17

This creates problems upstream.

matthewfeickert commented 3 years ago

six.moves was introduced in six v1.4.0 so

'six>=1.4.0'

should be added.