ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
16 stars 59 forks source link

feat: show optional log prefix function name #873

Closed gitcoindev closed 8 months ago

gitcoindev commented 8 months ago

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

netlify[bot] commented 8 months ago

Deploy Preview for ubiquibot-staging ready!

Name Link
Latest commit 78fc4a168990b9d0e11defa6881d5faec30f9c15
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 commented 8 months ago

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 commented 8 months ago

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 commented 8 months ago

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