Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,5 @@
# https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners#codeowners-syntax


# Unless specified, the jsteam is the default owner for nodejs repositories.
* @googleapis/spanner-client-libraries-nodejs @googleapis/jsteam
# Unless specified, the googleapis/cloud-sdk-nodejs-team is the default owner for nodejs repositories.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

For consistency with the syntax used for teams in CODEOWNERS files, it would be clearer to include the @ prefix for the team name in this comment.

# Unless specified, the @googleapis/cloud-sdk-nodejs-team is the default owner for nodejs repositories.

* @googleapis/spanner-client-libraries-nodejs @googleapis/cloud-sdk-nodejs-team

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

It's a good practice to end files with a newline character. This prevents potential issues with file concatenation and some command-line tools that process text files.

*     @googleapis/spanner-client-libraries-nodejs @googleapis/cloud-sdk-nodejs-team

Loading