Navigation Menu

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

Update codeowners #1831

Merged
merged 1 commit into from Jan 27, 2021
Merged

Conversation

travisgroth
Copy link
Contributor

Summary

Update default reviewers.

Related issues

n/a

Checklist

  • reference any related issues
  • updated docs
  • updated unit tests
  • updated UPGRADING.md
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@codeclimate
Copy link

codeclimate bot commented Jan 27, 2021

Code Climate has analyzed commit 87a3eaa and detected 0 issues on this pull request.

View more on Code Climate.

@codecov
Copy link

codecov bot commented Jan 27, 2021

Codecov Report

Merging #1831 (87a3eaa) into master (bec9805) will increase coverage by 0.0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1831   +/-   ##
======================================
  Coverage    56.6%   56.6%           
======================================
  Files         139     139           
  Lines       10178   10178           
======================================
+ Hits         5764    5766    +2     
+ Misses       3818    3816    -2     
  Partials      596     596           
Impacted Files Coverage Δ
internal/databroker/server.go 51.8% <0.0%> (+0.7%) ⬆️

@travisgroth travisgroth marked this pull request as ready for review January 27, 2021 16:21
@travisgroth travisgroth requested a review from a team as a code owner January 27, 2021 16:21
@travisgroth travisgroth merged commit 9423159 into pomerium:master Jan 27, 2021
@travisgroth travisgroth deleted the update_codeowners branch January 27, 2021 17:12
alexfornuto added a commit that referenced this pull request Aug 9, 2021
per pomerium-console/#1831
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.

None yet

3 participants