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

chore: update codeowners for 2fa #39442

Merged
merged 1 commit into from Jul 18, 2023
Merged

chore: update codeowners for 2fa #39442

merged 1 commit into from Jul 18, 2023

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Jul 17, 2023

Summary

The pull request #39411 modified lib/private/Authentication/TwoFactorAuth/Manager.php but did not request a review from the code owners.

TODO

  • Pull request after merge for a final test

Checklist

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb added the 3. to review Waiting for reviews label Jul 17, 2023
@kesselb kesselb self-assigned this Jul 17, 2023
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 18, 2023
@szaimen szaimen merged commit 61f0d07 into master Jul 18, 2023
36 of 37 checks passed
@szaimen szaimen deleted the 2fa-codeowners branch July 18, 2023 08:25
@szaimen szaimen added this to the Nextcloud 28 milestone Jul 18, 2023
@kesselb
Copy link
Contributor Author

kesselb commented Jul 18, 2023

Pull request after merge for a final test

Test PR: #39449

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants