Skip to content

Docs: fix docs dependencies#625

Merged
c24t merged 4 commits intoopen-telemetry:masterfrom
mauriciovasquezbernal:mauricio/fix-docs-dependencies
Apr 30, 2020
Merged

Docs: fix docs dependencies#625
c24t merged 4 commits intoopen-telemetry:masterfrom
mauriciovasquezbernal:mauricio/fix-docs-dependencies

Conversation

@mauriciovasquezbernal
Copy link
Copy Markdown
Member

  • Avoid duplicated dependency list in tox and docs-requirements
  • Sort list
  • Fix grpc duplicated entries

Preview available at https://opentelemetry-python-mauricio.readthedocs.io/en/mauricio-fix-docs-dependencies/.

@mauriciovasquezbernal mauriciovasquezbernal added the doc Documentation-related label Apr 28, 2020
@mauriciovasquezbernal mauriciovasquezbernal requested a review from a team April 28, 2020 18:31
Copy link
Copy Markdown
Contributor

@ocelotl ocelotl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Approving, just minor cosmetic changes requested.

@c24t c24t merged commit d01d779 into open-telemetry:master Apr 30, 2020
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this pull request Nov 1, 2020
* feat: introduce ended property on Span

Add a getter to Span and ReadableSpan to allow checking if the span is
ended without using a private field or rely on internals like
span.duration[0] < 0.

Refs: open-telemetry/opentelemetry-java#693
Refs: open-telemetry/opentelemetry-java#697

* chore: adapt after merge from master

* chore: revert changes in API

* chore: remove unneeded isEnded
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Documentation-related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants