Skip to content

Commit 5709008

Browse files
samimusallamFlarnahaddasbronfman
authored
fix(eslint-eqeqeq): updated the eqeqeq rule to match the core repo (open-telemetry#1485)
* fix(eslint-eqeqeq): updated eqeqeq from off to error * Revert "fix(eslint-eqeqeq): updated eqeqeq from off to error" This reverts commit b5ce8e5. * fix(eslint-eqeqeq): removed unnecessary changes * Update plugins/node/opentelemetry-instrumentation-aws-lambda/src/instrumentation.ts Co-authored-by: Gerhard Stöbich <[email protected]> --------- Co-authored-by: Gerhard Stöbich <[email protected]> Co-authored-by: Haddas Bronfman <[email protected]>
1 parent 7d31b69 commit 5709008

File tree

7 files changed

+11
-8
lines changed

7 files changed

+11
-8
lines changed

eslint.config.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@ module.exports = {
1212
},
1313
rules: {
1414
"quotes": ["error", "single", { "avoidEscape": true }],
15-
"eqeqeq": "off",
15+
"eqeqeq": [
16+
"error",
17+
"smart"
18+
],
1619
"prefer-rest-params": "off",
1720
"no-shadow": "off",
1821
"node/no-deprecated-api": ["warn"],

plugins/node/opentelemetry-instrumentation-aws-lambda/src/instrumentation.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ export class AwsLambdaInstrumentation extends InstrumentationBase {
132132
return moduleExports;
133133
},
134134
(moduleExports?: LambdaModule) => {
135-
if (moduleExports == undefined) return;
135+
if (moduleExports == null) return;
136136
diag.debug('Removing patch for lambda handler');
137137
this._unwrap(moduleExports, functionName);
138138
}

plugins/node/opentelemetry-instrumentation-aws-sdk/src/services/dynamodb.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ export class DynamodbServiceExtension implements ServiceExtension {
4141
),
4242
};
4343

44-
if (operation == 'BatchGetItem') {
44+
if (operation === 'BatchGetItem') {
4545
spanAttributes[SemanticAttributes.AWS_DYNAMODB_TABLE_NAMES] = Object.keys(
4646
normalizedRequest.commandInput.RequestItems
4747
);

plugins/node/opentelemetry-instrumentation-graphql/test/graphql.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -1179,7 +1179,7 @@ describe('graphql', () => {
11791179

11801180
it('should attach response hook data to the resulting spans', () => {
11811181
const querySpan = spans.find(
1182-
span => span.attributes[AttributeNames.OPERATION_TYPE] == 'query'
1182+
span => span.attributes[AttributeNames.OPERATION_TYPE] === 'query'
11831183
);
11841184
const instrumentationResult = querySpan?.attributes[dataAttributeName];
11851185
assert.deepStrictEqual(

plugins/node/opentelemetry-instrumentation-pino/src/instrumentation.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,13 @@ export class PinoInstrumentation extends InstrumentationBase {
4545
diag.debug(`Applying patch for pino@${moduleVersion}`);
4646
const instrumentation = this;
4747
const patchedPino = Object.assign((...args: unknown[]) => {
48-
if (args.length == 0) {
48+
if (args.length === 0) {
4949
return pinoModule({
5050
mixin: instrumentation._getMixinFunction(),
5151
});
5252
}
5353

54-
if (args.length == 1) {
54+
if (args.length === 1) {
5555
// eslint-disable-next-line @typescript-eslint/no-explicit-any
5656
const optsOrStream = args[0] as any;
5757
if (

propagators/opentelemetry-propagator-instana/src/InstanaPropagator.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ export class InstanaPropagator implements TextMapPropagator {
7070
setter.set(carrier, INSTANA_TRACE_ID_HEADER, spanContext.traceId);
7171
setter.set(carrier, INSTANA_SPAN_ID_HEADER, spanContext.spanId);
7272
const sampled =
73-
(spanContext.traceFlags & TraceFlags.SAMPLED) == TraceFlags.SAMPLED;
73+
(spanContext.traceFlags & TraceFlags.SAMPLED) === TraceFlags.SAMPLED;
7474
setter.set(carrier, INSTANA_LEVEL_HEADER, sampled ? '1' : '0');
7575
}
7676

propagators/opentelemetry-propagator-ot-trace/src/OTTracePropagator.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ export class OTTracePropagator implements TextMapPropagator {
8787

8888
extract(context: Context, carrier: unknown, getter: TextMapGetter): Context {
8989
let traceId = readHeader(carrier, getter, OT_TRACE_ID_HEADER);
90-
if (traceId.length == 16) traceId = `${PADDING}${traceId}`;
90+
if (traceId.length === 16) traceId = `${PADDING}${traceId}`;
9191
const spanId = readHeader(carrier, getter, OT_SPAN_ID_HEADER);
9292
const sampled = readHeader(carrier, getter, OT_SAMPLED_HEADER);
9393
const traceFlags =

0 commit comments

Comments
 (0)