Repository navigation
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
✅ PR checks passedThe linked issue has a description and all required Shipping project fields set. Thanks! |
|
|
@open-metadata maintainers: The PR metadata check is blocked because linked issue #27292 is not in the Shipping project. Could someone with Shipping project edit access add the issue and set Status, Source, Priority, Domain, and Release, then rerun PR Metadata Validation? My account has read-only access to this repository and cannot edit the project. Thanks! |
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
Code Review ✅ Approved🟢 Low risk · Removing two stale TODO comments in UserResource leaves bot authentication code and call sites unchanged, so runtime behavior and access decisions are unaffected. Removes two stale TODO comments from VerificationRules No rules evaluated Functional validation Not enabledAuto-approval Not enabledFor more verification details, comment OptionsDisplay: compact → Showing compact verification results. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |



Describe your changes:
Fixes #27292
Remove the two stale
TODO remove thiscomments aroundaddAuthMechanismToBotinUserResource.The method, its create and update call sites, and bot authentication behavior
remain unchanged.
Type of change:
High-level design:
Not applicable. This is a comment-only cleanup with no executable-code,
configuration, schema, or API changes.
Tests:
Use cases covered
addAuthMechanismToBotmethod remain unchanged.Unit tests
Not applicable. Removing comments cannot change runtime behavior.
Backend integration tests
Not applicable. No API or executable-code behavior changed.
Ingestion integration tests
Not applicable. No ingestion code changed.
Playwright (UI) tests
Not applicable. No UI code changed.
Validation performed
git diff --checkUI screen recording / screenshots:
Not applicable. No UI changes were made.
Checklist:
Fixes <issue-number>: <description>format.Fixes #27292.

Confidence Score: 5/5
This comment-only cleanup is safe to merge.Summary
Removes two stale TODO comments around
addAuthMechanismToBotinUserResource.java.Reviews (1) · Last reviewed commit: "Merge branch 'main' into fix/27292-remov..." · Reviewed by Greptile