[MNT] Migrate packaging to pyproject.toml#594
Conversation
|
Few Observations:
|
| ] | ||
|
|
||
| urls = { | ||
| Homepage = "https://github.com/manujosephv/pytorch_tabular" |
There was a problem hiding this comment.
Should it be "https://github.com/pytorch-tabular/pytorch_tabular" now? given the creation of the org?
There was a problem hiding this comment.
yes, I think so - although the old URL forwards to the new one, so it is not a breaking error. I will change this.
Fully agreed with all your points - I think they are already covered by this plan here? Where your statements are of course more granular than the general plan, as to how exactly we isolate, reduce, or update dependencies. For general plan, I think it is best if you comment on #592 to align on the precise steps? |
|
@phoeenniixx, are you able to start my workflows? |
I dont think so, Only maintainers can do so. I dont have write access to this repository. Workflows on my PR is running because I am a member of this org. |
|
I will check with @manujosephv on discord, I think he said we both have write access, but currently that does not seem to be the case? |
Migrates packaging to
pyproject.toml.requirementstopyproject.tomlsetup.pyandsetup.cfgtopyproject.tomlAlso carries out a minimal improvement of the testing workflow:
Depends on #597 which should be merged first