Browse Source

Review feedback

Craig Tiller 8 years ago
parent
commit
cbf5447a0d
3 changed files with 4 additions and 2 deletions
  1. 1 1
      src/core/ext/README.md
  2. 1 0
      src/core/ext/resolver/README.md
  3. 2 1
      src/core/lib/README.md

+ 1 - 1
src/core/ext/README.md

@@ -1,5 +1,5 @@
 Optional plugins for gRPC Core: Modules in this directory extend gRPC Core in
 Optional plugins for gRPC Core: Modules in this directory extend gRPC Core in
-useful ways.
+useful ways. All optional code belongs here.
 
 
 NOTE: The movement of code between lib and ext is an ongoing effort, so this
 NOTE: The movement of code between lib and ext is an ongoing effort, so this
 directory currently contains too much of the core library.
 directory currently contains too much of the core library.

+ 1 - 0
src/core/ext/resolver/README.md

@@ -1,3 +1,4 @@
 # Resolver
 # Resolver
 
 
 Implementations of various name resolution schemes.
 Implementations of various name resolution schemes.
+See the [naming spec](/doc/naming.md).

+ 2 - 1
src/core/lib/README.md

@@ -1,5 +1,6 @@
 Required elements of gRPC Core: Each module in this directory is required to
 Required elements of gRPC Core: Each module in this directory is required to
-build gRPC.
+build gRPC. If it's possible to envisage a configuration where code is not
+required, then that code belongs in ext/ instead.
 
 
 NOTE: The movement of code between lib and ext is an ongoing effort, so this
 NOTE: The movement of code between lib and ext is an ongoing effort, so this
 directory currently contains too much of the core library.
 directory currently contains too much of the core library.