[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

refactor(auth): unify how we handle requests in auth layer #10453

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

refactor(auth): unify how we handle requests in auth layer #10453

wants to merge 11 commits into from
+81 −132

Conversation

codyoss
Copy link
Member
@codyoss codyoss commented Jun 27, 2024

Doing this uniformly will allow us to add things like logging and telemetry more easily in the future.

@codyoss codyoss requested a review from a team as a code owner June 27, 2024 19:18
@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 27, 2024
@codyoss
refactor(auth): unify how we handle requests in auth layer
d885230 
Doing this uniformly will allow us to add things like logging and
telemetry more easily in the future.
@codyoss codyoss removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 27, 2024
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Jun 27, 2024
@codyoss codyoss added automerge Merge the pull request once unit tests and other checks pass. and removed automerge Merge the pull request once unit tests and other checks pass. labels Jun 28, 2024
@codyoss codyoss requested a review from noahdietz June 28, 2024 13:37
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 29, 2024
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@noahdietz noahdietz Awaiting requested review from noahdietz

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
automerge Merge the pull request once unit tests and other checks pass.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@codyoss