pyinfra-dev / pyinfra-dev/pyinfra
Cleanup state/inventory/config inter-references
Open
@Fizzadar is already working on this.
Since Sep 4, 2022.
API
spec
- Dominant language
- Python
- Stars
- 6k
- Forks
- 548
- Avg merge
- 7d 17h
- Merged PRs (30d)
- 13
Description
See: https://github.com/Fizzadar/pyinfra/pull/881/commits/947f8f691aec10e8ed963991db49afa36a58abc5
First thought move state.active_hosts (and other *_hosts variables) to inventory, since that's where they are accessed from anyway, this means Inventory.state reference can be dropped entirely.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.