Replace setup.py with pyproject.toml

[alifbe]

Solves #419

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c8c385b) 98.98% compared to head (86f1935) 98.98%.

❗ Current head 86f1935 differs from pull request most recent head 94de58a. Consider uploading reports for the commit 94de58a to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #420   +/-   ##
=======================================
  Coverage   98.98%   98.98%           
=======================================
  Files          16       16           
  Lines        2558     2558           
=======================================
  Hits         2532     2532           
  Misses         26       26           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mferrera]

Sorry for the delay! I'm approving so you can merge as needed but I think there are some things you could address if you wish

[mferrera]

Is this needed because this repository isn't src/ based?

[alifbe]

yes. I was tempted to move pyscal folder inside src But decided to use this instead. Any other benefit of using src/ based?

[mferrera]

src/ is preferred I guess because it very clearly indicates where to click if you want to look at the source code. One of those conventions that is pretty standard around many languages; in my opinion it is the better option. I think some tools (setuptools in particular, in this case) look for it first.

[alifbe]

Good point. I'll adopt this on next PR 👍