Skip to content
This repository has been archived by the owner on Dec 4, 2023. It is now read-only.

Add pyproject.toml support #120

Merged
merged 3 commits into from
Jul 12, 2022
Merged

Add pyproject.toml support #120

merged 3 commits into from
Jul 12, 2022

Conversation

cooperlees
Copy link
Contributor

  • [tool.ptr] Section support in pyproject.toml
  • Parses the toml into the same dict structure as setup.cfg + setup.py now
  • pyproject.toml is more preferred than setup.cfg + setup.py

Test:

  • Add unittest for parsing pyproject.toml
  • See Integration test stay working as pyproject.toml would be preferred
  • @patch mock to keep test_get_test_modules go through the if statements for coverage

Fixes #91

This does not change ptr to install using pyproject.toml yet ... just suport projects with ptr config in there.

- [tool.ptr] Section support in pyproject.toml
- Parses the toml into the same dict structure as `setup.cfg` + `setup.py` now
- `pyproject.toml` is more preferred than `setup.cfg` + `setup.py`

Test:
- Add unittest for parsing pyproject.toml
- See Integration test stay working as pyproject.toml would be preferred
- @patch mock to keep test_get_test_modules go through the if statements for coverage

Fixes #91

This does not change ptr to install using pyproject.toml yet ... just suport projects with ptr config in there.
@cooperlees cooperlees added the enhancement New feature or request label Jul 12, 2022
@cooperlees cooperlees requested review from aijayadams and fbnav July 12, 2022 05:16
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jul 12, 2022
@cooperlees cooperlees merged commit 8383d69 into main Jul 12, 2022
@cooperlees cooperlees deleted the pyproject_toml_support branch July 12, 2022 17:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support pyproject.toml cleanly
2 participants