Repository navigation
Add missing Node.js modules and remove EventStoreDB - #196
Conversation
Add Node.js entries for Azure Service Bus, InfluxDB, KurrentDB, Mosquitto, Oracle Free and S3Mock, which testcontainers-node has shipped without a catalog entry. Remove EventStoreDB. Its only entry was the Node.js module, which was renamed to KurrentDB, so its docs link returned 404.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
Summary by CodeRabbit
WalkthroughThe documentation pages for Azure Service Bus, InfluxDB, KurrentDB, Mosquitto, Oracle Free, and S3Mock gained Node.js entries and examples. The EventStoreDB documentation page was removed. ChangesModule documentation
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The new module examples and links match their published documentation, and the removed EventStoreDB entry is represented by KurrentDB. No actionable merge-blocking issue is evident. Pre-merge checks |
 |
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly summarizes the primary changes: adding missing Node.js module entries and removing EventStoreDB. |
| Description check | |
The description directly explains the six added Node.js entries, the EventStoreDB removal, and the reported verification steps. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
A rabbit browsed the module pages at night
Six new Node.js examples came into sight
With images and commands in place
One old page left the documentation space
The rabbit nibbled, then hopped out of sight
Comment @coderabbitai help to get the list of available commands.
HofmeisterAn
left a comment
HofmeisterAn
left a comment
There was a problem hiding this comment.
Oh I need to update this for .NET too.
Summary
Adds the Node.js entry to six modules that testcontainers-node ships but the catalog doesn't list for Node.js:
Each entry links the module's page on node.testcontainers.org and uses the image the module is tested against.
Removes EventStoreDB. Its only entry was the Node.js module, which was renamed to KurrentDB, so its docs link returned 404. KurrentDB now carries the Node.js entry.
Verification
find . -name 'index.md' | xargs yamllint --no-warnings -d .yamllintpasses, as in CI.