chore: fix typos and remove unwanted devDependency (open-telemetry#396)

Author: mayurkale22

packages/opentelemetry-core/package.json

@@ -23,6 +23,7 @@
   "keywords": [
     "opentelemetry",
     "nodejs",
+    "browser",
     "tracing",
     "profiling",
     "metrics",

packages/opentelemetry-plugin-grpc/test/grpc.test.ts

@@ -189,7 +189,7 @@ function startServer(grpc: GrpcModule, proto: any) {
   }
 
   server.addService(proto.GrpcTester.service, {
-    // An error is emited every time
+    // An error is emitted every time
     // request.num <= MAX_ERROR_STATUS = (status.UNAUTHENTICATED)
     // in those cases, erro.code = request.num
 

packages/opentelemetry-plugin-http/src/utils.ts

@@ -126,7 +126,8 @@ export const satisfiesPattern = <T>(
  * It will not re-throw exceptions from `list` provided by the client
  * @param constant e.g URL of request
  * @param [list] List of ignore patterns
- * @param [onException] callback for doing something when an exception has occured
+ * @param [onException] callback for doing something when an exception has
+ *     occurred
  */
 export const isIgnored = (
   constant: string,

packages/opentelemetry-scope-async-hooks/README.md

@@ -8,18 +8,18 @@ This package provides [async-hooks][async-hooks-doc] based scope manager which i
 
 ## What is a ScopeManager ?
 
-The definition and why they exist is available on [the readme of the socpe-base package][def-scope-manager].
+The definition and why they exist is available on [the readme of the scope-base package][def-scope-manager].
 
 ### Implementation in NodeJS
 
-NodeJS has a specific API to track async scope: [async-hooks][async-hooks-doc], it allows to track creation of new async operation and their respective parent. 
+NodeJS has a specific API to track async scope: [async-hooks][async-hooks-doc], it allows to track creation of new async operation and their respective parent.
 This package only handle storing a specific object for a given async hooks context.
 
 ### Limitations
 
 Even if the API is native to NodeJS, it doesn't cover all possible cases of scope propagation but there is a big effort from the NodeJS team to fix those. That's why we generally advise to be on the latest LTS to benefit from performance and bug fixes.
 
-There are known modules that break scope propagation ([somes of them are listed there][pkgs-that-break-ah]), so it's possible that the scope manager doesn't work with them.
+There are known modules that break scope propagation ([some of them are listed there][pkgs-that-break-ah]), so it's possible that the scope manager doesn't work with them.
 
 ### Prior arts
 
@@ -52,4 +52,4 @@ Apache 2.0 - See [LICENSE][license-url] for more information.
 [dd-js-tracer-scope]: https://github.com/DataDog/dd-trace-js/tree/master/packages/dd-trace/src/scope
 [opentracing-scope]: https://github.com/opentracing/opentracing-javascript/pull/113
 [diag-team-scope-discussion]: https://github.com/nodejs/diagnostics/issues/300
-[pkgs-that-break-ah]: https://github.com/nodejs/diagnostics/blob/master/tracing/AsyncHooks/problematic-modules.md
+[pkgs-that-break-ah]: https://github.com/nodejs/diagnostics/blob/master/tracing/AsyncHooks/problematic-modules.md

packages/opentelemetry-scope-base/package.json

@@ -17,6 +17,7 @@
   "keywords": [
     "opentelemetry",
     "nodejs",
+    "browser",
     "tracing",
     "profiling",
     "metrics",

packages/opentelemetry-shim-opentracing/README.md

@@ -6,7 +6,7 @@
 
 OpenTracing shim allows existing OpenTracing instrumentation to report to OpenTelemetry
 
-Note: Baggage is currently not progagated, see [issues/329](https://github.com/open-telemetry/opentelemetry-js/issues/329).
+Note: Baggage is currently not propagated, see [issues/329](https://github.com/open-telemetry/opentelemetry-js/issues/329).
 
 ## Installation
 

packages/opentelemetry-shim-opentracing/package.json

@@ -38,7 +38,6 @@
     "@types/mocha": "^5.2.7",
     "@types/node": "^12.6.9",
     "@opentelemetry/tracer-basic": "^0.1.0",
-    "@opentelemetry/scope-base": "^0.1.0",
     "codecov": "^3.5.0",
     "gts": "^1.1.0",
     "mocha": "^6.2.0",

packages/opentelemetry-shim-opentracing/test/Shim.test.ts

@@ -17,15 +17,12 @@
 import * as assert from 'assert';
 import * as opentracing from 'opentracing';
 import { BasicTracer, Span } from '@opentelemetry/tracer-basic';
-import { NoopScopeManager } from '@opentelemetry/scope-base';
 import { TracerShim, SpanShim, SpanContextShim } from '../src/shim';
 import { INVALID_SPAN_CONTEXT, timeInputToHrTime } from '@opentelemetry/core';
 import { performance } from 'perf_hooks';
 
 describe('OpenTracing Shim', () => {
-  const tracer = new BasicTracer({
-    scopeManager: new NoopScopeManager(),
-  });
+  const tracer = new BasicTracer();
   const shimTracer: opentracing.Tracer = new TracerShim(tracer);
   opentracing.initGlobalTracer(shimTracer);
 

packages/opentelemetry-types/package.json

@@ -17,6 +17,7 @@
   "keywords": [
     "opentelemetry",
     "nodejs",
+    "browser",
     "tracing",
     "profiling",
     "metrics",