Skip to content
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

chore: remove logger module #7298

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Conversation

bartoszherba
Copy link
Collaborator

@bartoszherba bartoszherba commented Oct 30, 2024

We do not need this module anymore.
closing: https://alokai.atlassian.net/browse/TFAL-92

@bartoszherba bartoszherba requested a review from a team as a code owner October 30, 2024 07:32
Copy link

changeset-bot bot commented Oct 30, 2024

⚠️ No Changeset found

Latest commit: 0e9934e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@bartoszherba bartoszherba requested review from dawid-ziobro and Fifciu and removed request for a team October 30, 2024 07:32
Copy link

sonarcloud bot commented Oct 30, 2024

@bartoszherba bartoszherba merged commit b9ce521 into feat/logger Oct 30, 2024
6 of 7 checks passed
@bartoszherba bartoszherba deleted the feat/remove-logger-module branch October 30, 2024 10:15
bartoszherba added a commit that referenced this pull request Nov 7, 2024
* feat: logger 1.0.0-rc0

* chore: add changeset

* ci: release (rc) (#7266)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.1

* ci: release (rc) (#7268)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger 1.0.0-rc.2

* ci: release (rc) (#7272)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add logger module to the SDK (#7267)

* ci: release (rc) (#7273)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger rc.3

* feat: rc.3

* ci: release (rc) (#7277)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: middleware logger (#7269)

* feat: injecting logger and metadata

* feat: injecting logger in hooks, onCreate, init fn, removed consola

* refactor: tidying, adding and adjusting tests

* refactor: applied CR tips

* chore: fix sdk test

* refactor: implemented CR suggestions

---------

Co-authored-by: Bartosz Herba <[email protected]>

* ci: release (rc) (#7278)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: add "class-methods-use-this": "off" rule (#7274)

* ci: release (#7275)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* CLOUD-3578 Improve Kubernetes Probes docs (#7264)

* CLOUD-3578 Improve readiness probe docs

* Update docs/content/3.middleware/2.guides/9.kubernetes-probes.md

Co-authored-by: Matt Maribojoc <[email protected]>

* Add changes from Kuba

---------

Co-authored-by: Matt Maribojoc <[email protected]>

* IN-3956 - Middleware extension docs is missing isNamespaced in the example code (#7271)

Co-authored-by: krystian wlodarski <[email protected]>

* feat: sdk logger wrapper (#7276)

* chore: update changeset

* ci: release (rc) (#7279)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: add CR fixes

* feat: middleware logger better logger scopes (#7281)

* fix: moved global logger error to non overwritable part of code

* chore: snapshot of done work

* refactor: MethodDecoratorManager

* refactor: reduced usage of singleton to support parallel calls of fns

* fix: infinite loop constraint on types

* Revert "fix: infinite loop constraint on types"

This reverts commit 01904e3.

* chore: bumped ts jest, and jest

* fix: test

* ci: release (rc) (#7284)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* docs: working with logger (#7285)

* fix: merging logger configs (#7286)

* docs(tfal-56): next/nuxt logger implementation (#7282)

* docs: update next/nuxt docs (#7290)

* chore: export types from logger module (#7292)

* ci: release (rc) (#7293)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: logger middleware error boundary (#7291)

* feat: error boundary

* refactor: refactor

* feat: use appropriate verbosity levels specifically in the browser environment (#7296)

* ci: release (rc) (#7297)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* chore: remove logger module (#7298)

* feat: block overwriting alokai metadata (#7300)

* feat: block overwriting alokai metadata

* refactor: outsourced deep merging of obj's to lodash.merge

* feat: guard to prevent overwriting alokai key in metadata obj

* fix: tests

* Update packages/middleware/src/logger/injectMetadata.ts

Co-authored-by: Bartosz Herba <[email protected]>

---------

Co-authored-by: Bartosz Herba <[email protected]>

* ci: release (rc) (#7303)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat: replace console with alokai logger in the middleware module (#7299)

* feat: change log level to verbosity (#7306)

* ci: release (rc) (#7307)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Bartosz Herba <[email protected]>

* docs(tfal-89,tfal-83): update logger docs for current solution (#7308)

* docs(tfal-89,tfal-83): update logger docs for current solution

* docs: fix

* docs: alokai metadata (#7305)

* docs: block overriding alokai key in provided metadata

* Apply suggestions from code review

Co-authored-by: daaf <[email protected]>

* Apply suggestions from code review

---------

Co-authored-by: daaf <[email protected]>
Co-authored-by: Bartosz Herba <[email protected]>

* chore: remove custom handler docs

* chore: exit pre mode

* feat: update logger exports and docs

* chore: update docs after cr

* docs: fix sdk docs

* feat: change log structured interface parameters

* Apply suggestions from code review - missing type annotation

Co-authored-by: daaf <[email protected]>

* chore: update node env for logger unit tests

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: daaf <[email protected]>
Co-authored-by: Filip Jędrasik <[email protected]>
Co-authored-by: Artur <[email protected]>
Co-authored-by: Matt Maribojoc <[email protected]>
Co-authored-by: Krystian Wlodarski <[email protected]>
Co-authored-by: krystian wlodarski <[email protected]>
Co-authored-by: Dawid Ziobro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants