Skip to content
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

Add support for tedious v18 #2266

Closed
kirrg001 opened this issue Jun 7, 2024 · 0 comments · Fixed by #2381 or #2375
Closed

Add support for tedious v18 #2266

kirrg001 opened this issue Jun 7, 2024 · 0 comments · Fixed by #2381 or #2375

Comments

@kirrg001
Copy link
Contributor

kirrg001 commented Jun 7, 2024

Hi there!

We have recently added support for v16 and v17.
Tedious released v18 four month ago.

I will try to create pull request.

Cheers

kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 7, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 10, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 10, 2024
kirrg001 added a commit to instana/opentelemetry-js-contrib that referenced this issue Jun 12, 2024
trentm added a commit to trentm/opentelemetry-js-contrib that referenced this issue Aug 10, 2024
tedious@18 adds its own .d.ts files. Compiling with them requires
TypeScript v5, and either target:'es2022' for 'AggregateError'
or using skipLibCheck:true in tsconfig.json.

This change limits the update to TypeScript v5 to just while
testing tedious@18 via test-all-versions' support for
'peerDependencies'.

THis also drops ts-mocha in favour of 'mocha --require ts-node/register'
as was done on opentelemetry-js.git in open-telemetry/opentelemetry-js#4840

Closes: open-telemetry#2266
@trentm trentm closed this as completed in 0e9791a Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment