Skip to content

Commit 9a387c7

Browse files
erikayao93dreamorosi
authored andcommitted
chore(logger): PowertoolsLogFormatter docstring and variable naming update (#1585)
* Updated formatAttributes for additional parameters and LogItem return type * Updated the unit tests to pass with new formatter * Updated Powertool named objects to Powertools * Updated tests to match new naming consistency * Updated for tests for new naming consistency * Updated formatter for new design decisions * Update Logger for ephemeral attributes * Update bringYourOwnFormatter documentation to match new formatter * Fixed incorrect return type, renamed variable for consistency
1 parent 4b12533 commit 9a387c7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Diff for: packages/logger/src/formatter/PowertoolsLogFormatter.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class PowertoolsLogFormatter extends LogFormatter {
1616
*
1717
* @param {UnformattedAttributes} attributes
1818
* @param {LogAttributes} additionalLogAttributes
19-
* @returns {PowertoolsLog}
19+
* @returns {LogItem}
2020
*/
2121
public formatAttributes(
2222
attributes: UnformattedAttributes,
@@ -36,11 +36,11 @@ class PowertoolsLogFormatter extends LogFormatter {
3636
xray_trace_id: attributes.xRayTraceId,
3737
};
3838

39-
const powertoolLogItem = new LogItem({ attributes: baseAttributes });
39+
const powertoolsLogItem = new LogItem({ attributes: baseAttributes });
4040

41-
powertoolLogItem.addAttributes(additionalLogAttributes);
41+
powertoolsLogItem.addAttributes(additionalLogAttributes);
4242

43-
return powertoolLogItem;
43+
return powertoolsLogItem;
4444
}
4545
}
4646

0 commit comments

Comments
 (0)