fix: Change addDependsOn to addDependency

[hlascelles]

addDependsOn is deprecated:

❯ yarn cdk synth
info: Preparing App...
[WARNING] aws-cdk-lib.CfnResource#addDependsOn is deprecated.
  use addDependency
  This API will be removed in the next major release.
[WARNING] aws-cdk-lib.CfnResource#addDependsOn is deprecated.
  use addDependency
  This API will be removed in the next major release.

[blimmer]

Hey @hlascelles - thanks for this PR. As you can see from the build failure, this change will also require an update to the minimum version of CDK v2. Currently, that's 2.1.0:

cdk-datadog-integration/.projenrc.js

Line 6 in e255fa0

cdkVersion: '2.1.0',

2.1.0 is quite old, so I think it's safe to bump to the minimum version of CDK that supports addDependency. You'll need to look in the changelogs to see when that was released and make the appropriate update.

Thanks for your contribution!

[hlascelles]

👍 Looks like v2.56.0 aws/aws-cdk@ecedb00

[blimmer]

Since this uses projen, you'll need to update the version in projenrc.js (linked in my comment above) and then run yarn projen. Commit the result and we can run the test suite. Thanks!

[hlascelles]

I struggled to do a yarn install as I get errors:

➤ YN0066: │ typescript@patch:typescript@npm%3A4.8.3#builtin<compat/typescript>::version=4.8.3&hash=8133ad: Cannot apply hunk #2 (set enableInlineHunks for details)
➤ YN0000: └ Completed in 11s 770ms
➤ YN0000: Failed with errors in 22s 702ms

Then if I try yarn set version latest I think I get further, but then yarn projen (with the new library version) gives Unknown Syntax Error: Unsupported option name ("--check-files").

I must admit this is all beyond my ken. I can't proceed this PR further, sorry!

[blimmer]

Fixed by #54 - I explained the changes over there. Thanks for your contribution! I'll close this PR in favor of the other one.