Skip to content

Commit de58eb6

Browse files
mhdawsonMylesBorins
authored andcommitted
meta: add N-API to codeowners coverage
We have this guidance for contributing to N-API: https://github.com/nodejs/node/blob/master/doc/guides/adding-new-napi-api.md It makes sense to have one of the N-API team sign off on commits that changes N-API Signed-off-by: Michael Dawson <michael_dawson@ca.ibm.com> PR-URL: #34039 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 8dd8b1a commit de58eb6

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

.github/CODEOWNERS

+7
Original file line numberDiff line numberDiff line change
@@ -74,3 +74,10 @@
7474
# ./lib/internal/modules/* @nodejs/modules
7575
# ./lib/internal/bootstrap/loaders.js @nodejs/modules
7676
# ./src/module_wrap* @nodejs/modules @nodejs/vm
77+
78+
# N-API
79+
80+
# /src/node_api* @nodejs/n-api
81+
# /src/js_native_api* @nodejs/n-api
82+
# /doc/guides/adding-new-napi-api.md @nodejs/n-api
83+
# /doc/api/n-api.md @nodejs/n-api

0 commit comments

Comments
 (0)