Skip to content

Conversation

@astuyve
Copy link
Contributor

@astuyve astuyve commented Oct 10, 2023

What does this PR do?

Specifies dependency on ddtrace < 2.0.0

Motivation

Users not utilizing our layer may bring their own version of ddtrace, which may be greater than our latest release.

Testing Guidelines

Additional Notes

Types of Changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)
@astuyve astuyve requested a review from a team as a code owner October 10, 2023 17:18
datadog = ">=0.41.0,<1.0.0"
wrapt = "^1.11.2"
ddtrace = "1.15.2"
ddtrace = "<2.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

There's also a reference to 1.15.2 in the poetry.lock file.

Copy link
Contributor

Choose a reason for hiding this comment

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

It seems you'll just need to run poetry install, see https://python-poetry.org/docs/basic-usage/#installing-with-poetrylock.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, added that already

@astuyve astuyve requested a review from zARODz11z October 11, 2023 19:42
@astuyve astuyve merged commit 2953548 into main Oct 16, 2023
@astuyve astuyve deleted the aj/pin-ddtrace-below-2 branch October 16, 2023 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants