Skip to content

docs(auto-instrument): add missing pip install in README #2061

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.

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

Merged
merged 1 commit into from
Aug 22, 2021

Conversation

blumamir
Copy link
Member

@blumamir blumamir commented Aug 22, 2021

Description

was following the auto-instrumentation example under docs. it is missing pip install flask to work correctly.

Fixes #2060 (issue)
add the pip install command to README

Type of change

  • This change requires a documentation update

How Has This Been Tested?

Run it locally on my fresh setup and got an error about "No module named 'flask'". After installing it as part of the setup the example wroks

Does This PR Require a Contrib Repo Change?

No

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@blumamir blumamir requested review from a team, codeboten and ocelotl and removed request for a team August 22, 2021 10:15
@srikanthccv srikanthccv added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 22, 2021
@owais owais merged commit aa1a2ac into open-telemetry:main Aug 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auto_instrumentation example not working
3 participants