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

Add CODEOWNERS #1042

Merged
merged 1 commit into from
Dec 9, 2019
Merged

Add CODEOWNERS #1042

merged 1 commit into from
Dec 9, 2019

Conversation

miekg
Copy link
Owner

@miekg miekg commented Dec 4, 2019

git shows:

git shortlog -sne | head
3311 Miek Gieben miek@miek.nl
157 Tom Thorogood me+github@tomthorogood.co.uk
39 Alex Sergeyev asergeyev@dyn.com
37 Andrew Tunnell-Jones andrew@tj.id.au
34 Filippo Valsorda filippo@cloudflare.com
21 Rafael Dantas Justo rafael@registro.br
19 Michael Haro mharo@google.com
14 Alex Sergeyev abc@alexsergeyev.com
13 chantra chantra@users.noreply.github.com
11 Alex Ciuba alexciuba@gmail.com

I took the top 2, but happy to extend this obvs.

git shows:

 git shortlog -sne | head
  3311	Miek Gieben <miek@miek.nl>
   157	Tom Thorogood <me+github@tomthorogood.co.uk>
    39	Alex Sergeyev <asergeyev@dyn.com>
    37	Andrew Tunnell-Jones <andrew@tj.id.au>
    34	Filippo Valsorda <filippo@cloudflare.com>
    21	Rafael Dantas Justo <rafael@registro.br>
    19	Michael Haro <mharo@google.com>
    14	Alex Sergeyev <abc@alexsergeyev.com>
    13	chantra <chantra@users.noreply.github.com>
    11	Alex Ciuba <alexciuba@gmail.com>

I took the top 2, but happy to extend this obvs.

Signed-off-by: Miek Gieben <miek@miek.nl>
@miekg
Copy link
Owner Author

miekg commented Dec 4, 2019

/cc @tmthrgd

I need to recompile caddy to setup dreck and then hook up the github app etc. But then you can use the following commands in issues and PRs https://github.com/miekg/dreck/blob/master/commands.md

@codecov-io
Copy link

codecov-io commented Dec 4, 2019

Codecov Report

Merging #1042 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1042      +/-   ##
=========================================
- Coverage   55.05%     55%   -0.06%     
=========================================
  Files          41      41              
  Lines        9879    9879              
=========================================
- Hits         5439    5434       -5     
- Misses       3419    3423       +4     
- Partials     1021    1022       +1
Impacted Files Coverage Δ
msg.go 76.83% <0%> (-0.67%) ⬇️
server.go 67.67% <0%> (-0.28%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22cda6d...0ea2a9e. Read the comment docs.

@tmthrgd
Copy link
Collaborator

tmthrgd commented Dec 8, 2019

@miekg Sounds good. Will commands.md remain the authoritative command list?

@miekg
Copy link
Owner Author

miekg commented Dec 9, 2019

Yes, I'm trying to keep that list up to date (I'm also ponder how to make folks aware of the bot in the first place)

@miekg miekg merged commit 6d0449f into master Dec 9, 2019
@miekg miekg deleted the owners branch December 9, 2019 11:50
aanm pushed a commit to cilium/dns that referenced this pull request Jul 29, 2022
git shows:

 git shortlog -sne | head
  3311	Miek Gieben <miek@miek.nl>
   157	Tom Thorogood <me+github@tomthorogood.co.uk>
    39	Alex Sergeyev <asergeyev@dyn.com>
    37	Andrew Tunnell-Jones <andrew@tj.id.au>
    34	Filippo Valsorda <filippo@cloudflare.com>
    21	Rafael Dantas Justo <rafael@registro.br>
    19	Michael Haro <mharo@google.com>
    14	Alex Sergeyev <abc@alexsergeyev.com>
    13	chantra <chantra@users.noreply.github.com>
    11	Alex Ciuba <alexciuba@gmail.com>

I took the top 2, but happy to extend this obvs.

Signed-off-by: Miek Gieben <miek@miek.nl>
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.

3 participants