8000 Enable log trace context injection by default by DarcyRaynerDD · Pull Request #44 · DataDog/datadog-lambda-python · GitHub
  • [go: up one dir, main page]
    Skip to content

    Enable log trace context injection by default #44

    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
    Merged
    DarcyRaynerDD merged 8 commits into from
    Feb 10, 2020
    Merged

    Enable log trace context injection by default #44

    DarcyRaynerDD merged 8 commits into from
    Feb 10, 2020
    +9 −6

    Conversation

    @DarcyRaynerDD
    Copy link
    Contributor
    @DarcyRaynerDD DarcyRaynerDD commented Feb 10, 2020

    What does this PR do?

    Enables trace log context injection by default. Also bumps the major version number since this is a breaking change.

    @DarcyRaynerDD DarcyRaynerDD requested a review from a team as a code owner February 10, 2020 16:54
    tianchu
    tianchu reviewed Feb 10, 2020
    View reviewed changes
    tianchu
    tianchu reviewed Feb 10, 2020
    View reviewed changes
    tianchu
    tianchu reviewed Feb 10, 2020
    View reviewed changes
    DarcyRaynerDD and others added 3 commits February 10, 2020 13:17
    @DarcyRaynerDD @tianchu
    Update README.md
    27accd0 
    Co-Authored-By: Tian Chu <tian.chu@datadoghq.com>
    @DarcyRaynerDD @tianchu
    Update README.md
    159150b 
    Co-Authored-By: Tian Chu <tian.chu@datadoghq.com>
    @tianchu
    Update README.md
    00c02c3
    @DarcyRaynerDD DarcyRaynerDD merged commit eeb1d02 into master Feb 10, 2020
    @DarcyRaynerDD DarcyRaynerDD deleted the darcy.rayner/set-log-injection-default branch February 10, 2020 18:44
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    @tianchu tianchu tianchu approved these changes

    Assignees

    No one assigned

    Labels

    None yet

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    3 participants

    @DarcyRaynerDD @tianchu
    0