Skip to content

Commit 9ca4b2a

Browse files
mmarchiniBethGriggs
authored andcommitted
build: add CODEOWNERS linter action
PR-URL: #34739 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
1 parent e54a684 commit 9ca4b2a

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

.github/CODEOWNERS

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
/lib/internal/net.js @nodejs/net
3939
/lib/internal/socket_list.js @nodejs/net
4040
/lib/internal/js_stream_socket.js @nodejs/net
41-
/src/cares_wrap.h @nodejs/net
41+
/src/cares_wrap.cc @nodejs/net
4242
/src/connect_wrap.* @nodejs/net
4343
/src/connection_wrap.* @nodejs/net
4444
/src/node_sockaddr* @nodejs/net
@@ -58,7 +58,6 @@
5858

5959
/deps/llhttp/* @nodejs/http @nodejs/net
6060
/doc/api/http.md @nodejs/http @nodejs/net
61-
/doc/api/http2.md @nodejs/http @nodejs/net
6261
/lib/_http_* @nodejs/http @nodejs/net
6362
/lib/http.js @nodejs/http @nodejs/net
6463
/lib/https.js @nodejs/crypto @nodejs/net @nodejs/http
@@ -68,7 +67,7 @@
6867
# http2
6968

7069
/deps/nghttp2/* @nodejs/http2 @nodejs/net
71-
/doc/api/http2.md @nodejs/http2 @nodejs/net
70+
/doc/api/http2.md @nodejs/http2 @nodejs/http @nodejs/net
7271
/lib/http2.js @nodejs/http2 @nodejs/net
7372
/lib/internal/http2/* @nodejs/http2 @nodejs/net
7473
/src/node_http2* @nodejs/http2 @nodejs/net

.github/workflows/linters.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,3 +77,11 @@ jobs:
7777
run: |
7878
make lint-py-build || true
7979
NODE=$(which node) make lint-py
80+
81+
lint-codeowners:
82+
runs-on: ubuntu-latest
83+
steps:
84+
- uses: actions/checkout@v2
85+
- uses: mszostok/codeowners-validator@v0.4.0
86+
with:
87+
checks: "files,duppatterns"

0 commit comments

Comments
 (0)