psydac icon indicating copy to clipboard operation
psydac copied to clipboard

Remove deprecated distutils (Python >=3.12)

Open kvrigor opened this issue 1 year ago • 1 comments

Solves first TODO item in #406. Summary of changes:

  • Removed finalize_options() since this method does not differ from its base class implementation, therefore making the override unnecessary.
  • For run(), it's sufficient to call the base class run() and then add the custom pyccel commands.
  • sub_run() should throw an error instead of failing silently (check=True).
  • Formatted code to conform with Pylint and PEP8 checks

kvrigor avatar May 08 '24 10:05 kvrigor

Can we do something about warning no. 3 at least?

Upon closer inspection it appers that this warning is not caused by Psydac, but rather by one of its dependencies. So please ignore this comment.

yguclu avatar May 08 '24 14:05 yguclu