Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

feat: show optional log prefix function name #873

Closed

Conversation

gitcoindev
Copy link
Contributor

Refactor the log function so that redundant prefixes passing in are removed. Those should be automatically derived from the stack trace.

The prefixes can be optionally enabled for any log function.

Resolves: #665

Quality Assurance (a few example function name prefixes added in square brackets):

image

Refactor the log function so that redundant prefixes passing in are removed.
Those should be automatically derived from the stack trace.

The prefixes can be optionally enabled for any log function.

Resolves: ubiquity#665
@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit 78fc4a1
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/65366dff36054f000818bfbf
😎 Deploy Preview https://deploy-preview-873--ubiquibot-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@0x4007
Copy link
Member

0x4007 commented Oct 23, 2023

Hey @gitcoindev as a heads up I've handled this and a ton of other small issues in my refactor I've been working on for some weeks now

I disabled /start on this repo until we can get this major refactor merged in (I was hoping to last week but there's just a lot of work.)

I plan to give credit to those who were midway on tasks without my prior notice that a lot are already completed.

@gitcoindev
Copy link
Contributor Author

Hi @pavlovcik thank you for the heads up, and for promising the credit. I hope your refactoring will be merged soon, until then I will wait with starting anything new in order not to clash with the incoming changes.

@0x4007
Copy link
Member

0x4007 commented Oct 24, 2023

I might need a reminder here after it's merged to give you credit.

@ubiquibot ubiquibot bot closed this Oct 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatic Stack Traces for Logger
2 participants