Skip to content

[Currency Bot] Bumped patch/minor dependencies - #1698

Merged
abhilash-sivan merged 3 commits into
mainfrom
currency-bot
Apr 23, 2025
Merged

[Currency Bot] Bumped patch/minor dependencies#1698
abhilash-sivan merged 3 commits into
mainfrom
currency-bot

Conversation

@instanacd

Copy link
Copy Markdown
Contributor

Tada!

@instanacd
instanacd requested a review from a team as a code owner April 23, 2025 04:10
@kirrg001

Copy link
Copy Markdown
Contributor

We will drop the pg updates because of brianc/node-postgres#3428. This might resolve the build error we see in the CI:

@instana/collector: Error: Cannot find module './lib/build-result'
@instana/collector: Require stack:
@instana/collector: - /artifacts/node_modules/pg-native/index.js
@instana/collector: - /artifacts/packages/collector/test/tracing/database/pg_native/app.js

@abhilash-sivan

Copy link
Copy Markdown
Contributor

Mongo db changelog:

https://www.mongodb.com/docs/drivers/node/current/whats-new/#std-label-version-6.16

Deprecates support for MongoDB Server 4.0. Support will be completely removed in a future minor release.

@kirrg001

Copy link
Copy Markdown
Contributor

Deprecates support for MongoDB Server 4.0. Support will be completely removed in a future minor release.

Seems like we are running our tests against v4 on Tekton.

image: mongo:4.1.13

I will create a PR to see if the tests are passing with latest.

@abhilash-sivan abhilash-sivan self-assigned this Apr 23, 2025
@abhilash-sivan
abhilash-sivan merged commit 528fd29 into main Apr 23, 2025
@brianc

brianc commented Apr 23, 2025

Copy link
Copy Markdown

Hey sorry about that. PR merged (thank you @brenoepics for the fix while I was sleeping!) and a new patch version of pg related modules released now. pg@8.15.2 should work. Please feel free to report any other issues you find!

@kirrg001
kirrg001 deleted the currency-bot branch April 23, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants