Initial list of best practices to follow in our project

[bogdandrutu]

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

[codecov]

Codecov Report

Merging #1925 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1925      +/-   ##
============================================
- Coverage     86.63%   86.61%   -0.02%     
- Complexity     1500     1502       +2     
============================================
  Files           184      184              
  Lines          5663     5664       +1     
  Branches        582      582              
============================================
  Hits           4906     4906              
  Misses          551      551              
- Partials        206      207       +1     

Impacted Files	Coverage Δ	Complexity Δ
...ain/java/io/opentelemetry/api/baggage/Baggage.java	100.00% <100.00%> (ø)	6.00 <2.00> (+2.00)
...ava/io/opentelemetry/api/baggage/BaggageUtils.java	100.00% <100.00%> (ø)	6.00 <5.00> (-1.00)
...io/opentelemetry/api/baggage/ImmutableBaggage.java	96.77% <100.00%> (ø)	17.00 <0.00> (ø)
.../api/baggage/propagation/W3CBaggagePropagator.java	93.18% <100.00%> (ø)	15.00 <0.00> (ø)
.../io/opentelemetry/opentracingshim/Propagation.java	100.00% <100.00%> (ø)	5.00 <0.00> (ø)
...lemetry/context/PersistentHashArrayMappedTrie.java	44.54% <0.00%> (-0.91%)	4.00% <0.00%> (ø%)
...ain/java/io/opentelemetry/context/LazyStorage.java	88.46% <0.00%> (ø)	9.00% <0.00%> (ø%)
.../java/io/opentelemetry/context/DefaultContext.java	100.00% <0.00%> (ø)	7.00% <0.00%> (ø%)
...va/io/opentelemetry/opentracingshim/TraceShim.java	100.00% <0.00%> (ø)	4.00% <0.00%> (+1.00%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 604aee2...726ff76. Read the comment docs.

[jkwatson]

good start!