Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wt 13481 access check poc prefix #skip-ci #11039

Closed
wants to merge 4 commits into from

Conversation

ershov
Copy link
Contributor

@ershov ershov commented Sep 19, 2024

#skip-ci

Copy link

Thanks for creating a pull request! The below questions and checklist are intended to help with verifying your change is well tested. Response is optional, but if you choose to respond please edit this comment.

What makes this change safe?

A good answer to this question helps the reviewers understand where they should focus their attention, so please consider these questions:

  • Is the change risky or not? Why?
  • What tests are you adding or changing? Why?
  • What existing tests are you relying on?
  • What, if anything, are you concerned about that you'd like the reviewer to focus on?
    References:
  • Risk level guide
  • Testing frameworks

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have made corresponding changes to the documentation (if applicable).
  • I have added/updated tests that demonstrate my fix is effective or that my feature works correctly.

@ershov ershov force-pushed the wt-13481-access-check-poc-prefix branch from 9685772 to a5a6a4d Compare September 19, 2024 21:36
@ershov ershov closed this Sep 19, 2024
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.

1 participant