Skip to content

✨ (custom logger) add option to supply custom logger#72

Merged
maybeanerd merged 6 commits intobotblock:masterfrom
MeerBiene:master
Apr 8, 2021
Merged

✨ (custom logger) add option to supply custom logger#72
maybeanerd merged 6 commits intobotblock:masterfrom
MeerBiene:master

Conversation

@MeerBiene
Copy link
Copy Markdown
Contributor

Closes #71

Copy link
Copy Markdown
Member

@maybeanerd maybeanerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the setup in general, but specifically the typing left me with some questions as can be seen in the comments.

edit:
oh and also, why does this PR change the package-lock and yarn-lock? Did we have outdated lockfiles in the past commit?

Comment thread src/blapi.ts Outdated
Comment thread src/blapi.ts Outdated
Comment thread src/blapi.ts Outdated
Comment thread src/bttps.ts Outdated
Comment thread src/bttps.ts Outdated
@maybeanerd maybeanerd added the enhancement New feature or request label Apr 2, 2021
@maybeanerd maybeanerd self-assigned this Apr 2, 2021
@MeerBiene
Copy link
Copy Markdown
Contributor Author

I like the setup in general, but specifically the typing left me with some questions as can be seen in the comments.

edit:
oh and also, why does this PR change the package-lock and yarn-lock? Did we have outdated lockfiles in the past commit?

Oh boi here we go again, tbh I didn't realize they changed, I just hacked my changes in and pushed it. I have no idea why they changed tho

@advaith1 advaith1 changed the title ✨ (custom logger) add option to supply custom logger ✨ (custom logger) add option to supply custom logger Apr 3, 2021
@maybeanerd maybeanerd self-requested a review April 7, 2021 07:57
Copy link
Copy Markdown
Member

@maybeanerd maybeanerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some more questions, but in general no huge issues, just some optimization/cleanup

Comment thread README.md Outdated
Comment thread src/blapi.ts Outdated
Comment thread src/bttps.ts
Comment thread src/bttps.ts Outdated
Comment thread src/blapi.ts Outdated
Comment thread src/blapi.ts Outdated
at some point I want to just replace this with all contributors (https://allcontributors.org/docs/en/overview), but as long as this file exists, ill make sure its up to date
@maybeanerd maybeanerd merged commit 02491fc into botblock:master Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bring your own logger

2 participants