search

cldf-clts / clts-legacy

Cross-Linguistic Transcription Systems
Apache License 2.0
4 stars 3 forks source link

UnicodeMixin has been deprecated #135

Closed Anaphory closed 5 years ago

Anaphory commented 5 years ago

The UnicodeMixin class used in

https://github.com/cldf/clts/blob/80ff1a55d025541d6e5f6c6cea37e2c26fa82510/src/pyclts/models.py#L52

has been deprecated and should not be used any more.

xrotwang commented 5 years ago

Yes, we'll fix this when dropping py2 support.

Anaphory commented 5 years ago

Ah, okay. I had (too) many warnings sweep by in my travis build of LexiRumah, so I created this and then made a hack to my travis config to ignore those deprecation warnings. (Can the warnings bit be made to output the warning only once?)

xrotwang commented 5 years ago

Yes, it can. See the setup.py in pyglottolog

Gereon Kaiping notifications@github.com schrieb am Mi., 18. Sep. 2019, 15:49:

Is there a way this problem can be solved by putting the right requirements in setup.py? Obviously for py>3.5, it does not have to pin to clldutils<3.0.0, but for earlier python it does – can that logic be expressed and would it be useful?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cldf/clts/issues/135?email_source=notifications&email_token=AAGUOKFKHJPIVNSIJNXQDY3QKIWU5A5CNFSM4IX6S7PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7AD6OY#issuecomment-532692795, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGUOKCNPP7LWTAQSDSBXE3QKIWU5ANCNFSM4IX6S7PA .

xrotwang commented 5 years ago

See https://github.com/cldf-clts/pyclts