Skip to content

feat(logger): basic logger logic #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 27 commits into from
Mar 14, 2021
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
56ac31d
feat(logger): basic logger logic
saragerion Feb 22, 2021
251e66b
feat(logger): logger class, log schema, config service
saragerion Mar 4, 2021
380e678
refactor(logger): rename sample rate and log event enabled variable
saragerion Mar 4, 2021
fcc55fa
chore(logger): update linting rules
saragerion Mar 10, 2021
9989d08
chore(logger): ignore dummy resources in linting rules
saragerion Mar 10, 2021
5e0e117
chore(tests): add dummy Lambda context, custom lambda event
saragerion Mar 10, 2021
f896ba9
chore(logger): include examples folder in compiler options
saragerion Mar 10, 2021
9ce50c0
refactor(logger): folders structure, logger logic, types
saragerion Mar 10, 2021
8c9601a
fix(logger): default attributes for child logger
saragerion Mar 10, 2021
410cb5d
refactor(logger): remove unnecessary customConfigService parameters i…
saragerion Mar 10, 2021
2077dcd
chore(logger): remove sort members linting rule
saragerion Mar 10, 2021
bd13a28
refactor(logger): simplified formatter and logger logic, update examp…
saragerion Mar 10, 2021
7cc2111
refactor(logger): remove redundant constructor in config classes
saragerion Mar 10, 2021
c359e54
fix(logger): use UnformattedAttributes as input type for formatting
saragerion Mar 10, 2021
48f04cd
fix(logger): environment attributes, polishing
saragerion Mar 10, 2021
7f4463b
chore(logger): set logger metadata as default attributes in examples
saragerion Mar 10, 2021
e106e5e
refactor(logger): add context enabled variable
saragerion Mar 12, 2021
471bebe
chore(logger): update test run
saragerion Mar 12, 2021
1350bee
build(logger): test workload - add step to install node modules in pa…
saragerion Mar 12, 2021
52f03c6
fix(logger): broken GH action for test
saragerion Mar 12, 2021
88a2903
build(logger): edit npm scripts
saragerion Mar 12, 2021
9578d30
build(logger): fix GA yaml
saragerion Mar 12, 2021
c68f1ff
chore(logger): temporarily lowered logger test coverage
saragerion Mar 12, 2021
d19a267
chore(logger): temporarily lowered logger test coverage for lines
saragerion Mar 12, 2021
9afefad
chore(logger): move aws mock events to lambda-events branch, GH-10
saragerion Mar 13, 2021
02f6aef
docs(logger): add usage with code examples
saragerion Mar 13, 2021
2c12121
refactor(logger): error input handling, powertoolAttributes, customAt…
saragerion Mar 13, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
69 changes: 45 additions & 24 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,42 +1,63 @@
module.exports = {
env: {
'jest': true,
'browser': false,
'es2020': true,
'jest': true,
'node': true,
'es2020': true
},
parser: '@typescript-eslint/parser',
extends: [
'plugin:@typescript-eslint/eslint-recommended',
'plugin:@typescript-eslint/recommended'
],
ignorePatterns: ['tests/resources/*'],
parser: '@typescript-eslint/parser',
plugins: ['@typescript-eslint'],
rules: {
'no-console': 0,
'semi': [ 'error', 'always' ],
'newline-before-return': 2,
'indent': [ 'error', 2, { 'SwitchCase': 1 } ],
'quotes': [ 'error', 'single', { 'allowTemplateLiterals': true } ],
'object-curly-spacing': [ 'error', 'always' ],
'@typescript-eslint/ban-ts-ignore': ['off'],
'@typescript-eslint/camelcase': ['off'],
'@typescript-eslint/explicit-function-return-type': [ 'error', { 'allowExpressions': true } ],
'@typescript-eslint/explicit-member-accessibility': 'error',
'@typescript-eslint/indent': [ 'error', 2, { 'SwitchCase': 1 } ],
'@typescript-eslint/interface-name-prefix': ['off'],
'@typescript-eslint/member-delimiter-style': [ 'error', { 'multiline': { 'delimiter': 'none' } } ],
'@typescript-eslint/member-ordering': [ 'error', {
'default': { 'memberTypes': [
'signature',
'public-field', // = ["public-static-field", "public-instance-field"]
'protected-field', // = ["protected-static-field", "protected-instance-field"]
'private-field', // = ["private-static-field", "private-instance-field"]
'constructor',
'public-method', // = ["public-static-method", "public-instance-method"]
'protected-method', // = ["protected-static-method", "protected-instance-method"]
'private-method' // = ["private-static-method", "private-instance-method"]
] ,
'order': 'alphabetically' }
} ],
'@typescript-eslint/no-explicit-any': 'error',
'@typescript-eslint/no-inferrable-types': ['off'],
'@typescript-eslint/no-unused-vars': [ 'error', { 'argsIgnorePattern': '^_' } ],
'@typescript-eslint/no-use-before-define': ['off'],
'@typescript-eslint/semi': [ 'error', 'always' ],
'array-bracket-spacing': [ 'error', 'always', { 'singleValue': false } ],
'arrow-body-style': [ 'error', 'as-needed' ],
'computed-property-spacing': [ 'error', 'never' ],
'no-multiple-empty-lines': [ 'error', { 'max': 1, 'maxBOF': 0 } ],
'prefer-arrow-callback': 'error',
'func-style': [ 'warn', 'expression' ],
'no-multi-spaces': [ 'error', { 'ignoreEOLComments': false } ],
'indent': [ 'error', 2, { 'SwitchCase': 1 } ],
'keyword-spacing': 'error',
'@typescript-eslint/semi': [ 'error', 'always' ],
'@typescript-eslint/indent': [ 'error', 2, { 'SwitchCase': 1 } ],
'@typescript-eslint/explicit-function-return-type': [ 'error', { 'allowExpressions': true } ],
'@typescript-eslint/member-delimiter-style': [ 'error', { 'multiline': { 'delimiter': 'none' } } ],
'@typescript-eslint/interface-name-prefix': ['off'],
'@typescript-eslint/camelcase': ['off'],
'@typescript-eslint/no-use-before-define': ['off'],
'@typescript-eslint/ban-ts-ignore': ['off'],
'@typescript-eslint/no-inferrable-types': ['off'],
'@typescript-eslint/no-unused-vars': [ 'error', { 'argsIgnorePattern': '^_' } ],
'@typescript-eslint/no-explicit-any': 'error',
'@typescript-eslint/explicit-member-accessibility': 'error'
'newline-before-return': 2,
'no-console': 0,
'no-multi-spaces': [ 'error', { 'ignoreEOLComments': false } ],
'no-multiple-empty-lines': [ 'error', { 'max': 1, 'maxBOF': 0 } ],
'object-curly-spacing': [ 'error', 'always' ],
'prefer-arrow-callback': 'error',
'quotes': [ 'error', 'single', { 'allowTemplateLiterals': true } ],
'semi': [ 'error', 'always' ],
'sort-imports': [ 'error', {
'allowSeparatedGroups': true,
'ignoreCase': true,
'ignoreDeclarationSort': false,
'ignoreMemberSort': true,
'memberSyntaxSortOrder': [ 'all', 'single', 'multiple', 'none' ]
} ]
}
};
9 changes: 6 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -11,11 +11,14 @@ jobs:
steps:
- uses: actions/checkout@v2
- name: Install packages
run: npm install
run: |
export NODE_ENV=dev
npm ci
npm run lerna-ci
- name: Run lint
run: npm run lint
run: npm run lerna-lint
- name: Run tests
run: npm run test
run: npm run lerna-test
- name: Report Coverage
if: ${{ github.event_name == 'pull_request' }}
uses: romeovs/[email protected]
13 changes: 13 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 10 additions & 9 deletions package.json
Original file line number Diff line number Diff line change
@@ -6,15 +6,15 @@
"types": "lib/",
"scripts": {
"commit": "commit",
"ci": "lerna exec -- npm ci",
"test": "lerna exec -- jest --coverage --detectOpenHandles",
"build": "lerna exec -- tsc",
"lint": "lerna exec -- eslint \"./{src,tests}/**/*.ts\"",
"format": "lerna exec -- eslint --fix \"./{src,tests}/**/*.ts\"",
"prepare": "lerna exec -- npm run build",
"prepublishOnly": "lerna exec -- npm test && lerna exec -- npm run lint",
"preversion": "lerna exec -- npm run lint",
"version": "lerna exec -- npm run format && git add -A src",
"lerna-ci": "lerna exec -- npm ci",
"lerna-test": "lerna exec -- jest --coverage --detectOpenHandles",
"lerna-build": "lerna exec -- tsc",
"lerna-lint": "lerna exec -- eslint \"./{src,tests}/**/*.ts\"",
"lerna-format": "lerna exec -- eslint --fix \"./{src,tests}/**/*.ts\"",
"lerna-prepare": "lerna exec -- npm run build",
"lerna-prepublishOnly": "lerna exec -- npm test && lerna exec -- npm run lint",
"lerna-preversion": "lerna exec -- npm run lint",
"lerna-version": "lerna exec -- npm run format && git add -A src",
"postversion": "git push && git push --tags"
},
"repository": {
@@ -34,6 +34,7 @@
"devDependencies": {
"@commitlint/cli": "^11.0.0",
"@commitlint/config-conventional": "^11.0.0",
"@types/aws-lambda": "^8.10.72",
"@types/jest": "^26.0.19",
"@types/node": "^14.14.16",
"@typescript-eslint/eslint-plugin": "^4.11.1",
Loading