maelstrom
maelstrom copied to clipboard
Allow for use of pyproject.toml for maelstrom-pytest config
It would be really nice to not have to require an additional config item for use of this project. pytest already handles support of the pyproject.toml found here and follows the guidance in PEP 621.
More than willing to help with contribution/development here but wanted to ensure that this is feasible and/or wanted before spending the time!
Thanks for opening the issue.
More than willing to help with contribution/development here but wanted to ensure that this is feasible and/or wanted before spending the time!
As for feasibility of this I think this will work. All of our configuration for tests (what we store in maelstrom-pytest.toml which is the project-specific configuration) is just toml and it seems like the pyproject.toml file allows toml which can be interpreted by individual tools. I wasn't able to 100% verify with my limited research I did just now, but if you know more about it maybe you can affirm this. If it lets us put whatever toml we need in it, and that doesn't interfere with the configuration or usage of the file by other tools, then I see no reason why we can't just read up the configuration from there.
As for what we think about the feature, I think that this is a good feature, less configuration files is great and it seems to me like a lot of Python tools have standardized around using this file.
As for completion of this feature, I'm not sure where we would put this in priority with other things, but if you want to drive development and contribute that we would be thrilled. I can assign this to you and I encourage you to join our discord. I would be more than happy to help you write and contribute this code. Otherwise, we will get to this eventually. Let me know what you think.
I believe this should be able to work and I am open to work on it (just need to free up some time to do so)! You should be able to just specify that you are a "tool" in the pyproject.toml file and go from there!