open-telemetry / open-telemetry/opentelemetry-python
infra: evaluate toml dependency in eachdist scripts
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 2.6k
- Forks
- 1k
- Avg merge
- 4d 15h
- Merged PRs (30d)
- 19
Description
Description
https://github.com/open-telemetry/opentelemetry-python/pull/4099 adds a dependency in eachdist.py for toml.
In https://github.com/open-telemetry/opentelemetry-python/pull/4265 we have toml installed in CI as temporary solution to avoid issues with the release.
We need to evaluate if we want to move forward with that.
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.
Research direction
Start by reading eachdist.py and the changes in PR 4099 that introduced the toml dependency. Review PR 4265 and its temporary CI installation, then determine whether the dependency should remain and update the relevant release or CI configuration; done means the dependency decision is implemented consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- build-system, release
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100