delph-in / pydelphin

Python libraries for DELPH-IN
https://pydelphin.readthedocs.io/
MIT License
77 stars 27 forks source link

Fix #360: remove implicit optional types #361

Closed goodmami closed 1 year ago

goodmami commented 1 year ago

Fixing this in PyDelphin touched a lot of files but most changes were trivial. Along the way there were some typing gaps discovered and further work will be needed to fix some of these (e.g., the subclassing of MRS, EDS, and DMRS from the SemanticStructure violates the Liskov Substitution Principle on some methods).

This PR also restores the more strict Mypy checks in CI.