Skip to content

BOX-210 adapt all logs to pino #60

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.

Sign up for GitHub

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

Jump to bottom
Merged
sergeysova merged 3 commits into from
Aug 18, 2021
Merged

BOX-210 adapt all logs to pino #60

sergeysova merged 3 commits into from
Aug 18, 2021

Conversation

@sergeysova
Copy link
Member

@sergeysova sergeysova commented Aug 17, 2021

Branch name

  • Your branch should be prefixed with feature/ or fix/ or chore/ or refactor/ or tests/
  • Add issue identifier
  • Add 3-5 words of description separated with -

Example:

feature/BOX-12-add-login-form
fix/BOX-8-loading-panel-on-first-load
chore/BOX-22-add-release-drafter

PullRequest

  • This PullRequest implements new feature, fix bug, or some other changes
  • If PR is not ready to review mark it as Draft
  • All commits in this PR should be created by yarn commit by conventional-commits
  • Title of the PullRequest should have issue(s) identifier(s) ("BOX-123 Example title of PR")

@Drevoed Drevoed self-requested a review August 18, 2021 11:08
@sergeysova sergeysova merged commit 725aa93 into master Aug 18, 2021
@sergeysova sergeysova deleted the feature/box-210-adapt-all-logs-to-pino-also-known-as branch August 18, 2021 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Drevoed Drevoed Drevoed approved these changes

Assignees

No one assigned

Labels

Infrastructure

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sergeysova @Drevoed