Skip to content

Solvefix - #667

Open
69starman wants to merge 2 commits into
TrusTrove:mainfrom
69starman:solvefix
Open

Solvefix#667
69starman wants to merge 2 commits into
TrusTrove:mainfrom
69starman:solvefix

Conversation

@69starman

Copy link
Copy Markdown
Contributor

CLOSES #610

Linked Issue

Closes #

Summary

Changes

Test Coverage

  • Positive path test added
  • Negative path test added (where applicable)
  • All existing tests pass

Checklist

  • cargo fmt --all --check passes
  • cargo clippy --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Documentation updated (if applicable)
  • CHANGELOG.md updated (if applicable)

Screenshots

N/A

@69starman
69starman requested a review from K1NGD4VID as a code owner August 26, 2026 09:32
@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@69starman Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

mergekeeper[bot]
mergekeeper Bot previously approved these changes Aug 26, 2026

@mergekeeper mergekeeper Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Maintainer-authorized dependency-bot merge.

@mergekeeper

mergekeeper Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

MergeKeeper merge status

Status: blocked
PR state: open
Mergeability: mergeable
Checked commit: 0f1b85b38fb476b253dd4e3f8a8b31ed617d8436.

Reason:
GitHub pull request merge request failed with 403: Resource not accessible by integration

Next steps:

  1. Check the linked GitHub status and repository-rule details.
  2. Fix the reported issue or update the repository rule.
  3. Push a new commit or retry after the requirement becomes eligible.

@69starman
69starman dismissed mergekeeper[bot]’s stale review September 6, 2026 19:42

The merge-base changed after approval.

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.

refactor(registry): admin-lookup-and-require_auth block is duplicated across six functions

1 participant