Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Documentation mentions eachdist #1994

Closed
ocelotl opened this issue Jul 27, 2021 · 2 comments
Closed

Documentation mentions eachdist #1994

ocelotl opened this issue Jul 27, 2021 · 2 comments
Assignees
@ocelotl
Labels
triaged

Comments

@ocelotl
Copy link
Contributor

ocelotl commented Jul 27, 2021

Our documentation suggests our contributors should use scripts/eachdist.py. As mentioned in #1462, this is problematic because eachdist.pyis a tool that overlaps withtox`. Users have already reported some trouble, like in #1846.

Right now tox calls eachdist.py. Ideally we should remove eachdist.py completely, but since this requires some refactoring of tox.ini, at least remove it from the documentation so that users are not confused.
@ocelotl ocelotl self-assigned this Jul 27, 2021
ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue Jul 27, 2021
@ocelotl
Remove eachdist from documentation
58c84a5 
Fixes open-telemetry#1994
@ocelotl ocelotl mentioned this issue Jul 27, 2021
Closed
@github-actions
Copy link

This issue was marked stale due to lack of activity. It will be closed in 30 days.

@ocelotl
Copy link
Contributor Author

ocelotl commented Nov 17, 2022

Our documentation no longer suggests scripts/eachdist.py be used, closing.

@ocelotl ocelotl closed this as completed Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ocelotl ocelotl

Labels
triaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove eachdist from documentation ocelotl/opentelemetry-python
2 participants
@ocelotl @srikanthccv