Repository navigation
docs(collections): explain connector cleanup on deletion - #340
adityamishra-hydradb wants to merge 2 commits into
Conversation
Signed-off-by: adityamishra-hydradb <321510987+adityamishra-hydradb@users.noreply.github.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
✅ Mintlify HygieneNo issues found. |
✅ OpenHack SummarySecurity review of docs(collections): explain connector cleanup on deletion. 1 changed file; 0 findings at or above the low reporting threshold. Confidence Score: 5/5No reportable security findings were detected in this scan. Security merge-readiness rubric: 1 = critical, 2 = high, 3 = medium, 4 = low, 5 = no reportable findings. This score reflects scan findings, not a guarantee of correctness or complete coverage. Files Needing Attention: None Important Files Changed
Last reviewed commit: 9a4bd9e · View review on OpenHack
|
[Low impact] The documentation change appears safe to merge after the application change reaches production. SummaryUpdates the delete-collection reference to say that deletion also disables matching connector default routes.
Reviews (3) · Last reviewed commit: "docs(collections): clarify mixed connect..." · Reviewed by Greptile |
|
@greptileai review |
|
/devin review |
|
/devin review |
|
@greptileai review |
1 similar comment
|
@greptileai review |
|
/devin review |
|
Superseded by #342. The replacement contains one DCO-signed commit and avoids a prohibited force-push. |
For humans
The delete-collection reference now states that deletion removes dedicated connectors and only matching resources from mixed connectors.
Linear: https://linear.app/hydradb/issue/ENG-3301
Merge this PR only after the application change reaches production.
For agents
82850ca5440e56a8a967f147cf6768659ac6ba80frommain.npm run hygiene:strictnpm run verify:agents-endpointsnpx mintlify validateverify:agents-endpointschecked 54 endpoint mentions.Decisions taken without review
Session Details
(aside)to your comment to have me ignore it.Need help on this PR? Tag
@codesmith-botwith what you need.