Merge pull request #401 from humblec/new-changes

Update coding.md
This commit is contained in:
Humble Devassy Chirammal 2019-06-04 12:06:05 +05:30 committed by GitHub
commit 0e49ea90f3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 5 deletions

View File

@ -5,7 +5,7 @@ Card](https://goreportcard.com/badge/github.com/ceph/ceph-csi)](https://goreport
[![Build [![Build
Status](https://travis-ci.org/ceph/ceph-csi.svg?branch=master)](https://travis-ci.org/ceph/ceph-csi) Status](https://travis-ci.org/ceph/ceph-csi.svg?branch=master)](https://travis-ci.org/ceph/ceph-csi)
This repo containes [Container Storage Interface(CSI)] This repo contains [Container Storage Interface(CSI)]
(<https://github.com/container-storage-interface/>) driver, provisioner, (<https://github.com/container-storage-interface/>) driver, provisioner,
and attacher for Ceph RBD and CephFS. and attacher for Ceph RBD and CephFS.

View File

@ -15,8 +15,6 @@ the code and will be pointed out in the review process:
* Keep variable names short for variables that are local to the function. * Keep variable names short for variables that are local to the function.
* Do not export a function or variable name outside the package until you * Do not export a function or variable name outside the package until you
have an external consumer for it. have an external consumer for it.
* Do not use named return values in function definitions. Use only the type.
Exception: defer()'d functions.
### Imports ### Imports
@ -60,8 +58,6 @@ import (
### Logging ### Logging
* If a function is only invoked as part of a transaction step, always use the
transaction's logger to ensure propagation of request ID and transaction ID.
* The inner-most utility functions should never log. Logging must almost always * The inner-most utility functions should never log. Logging must almost always
be done by the caller on receiving an `error`. be done by the caller on receiving an `error`.
* Always use log level `DEBUG` to provide useful **diagnostic information** to * Always use log level `DEBUG` to provide useful **diagnostic information** to