ubiquity / ubiquibot

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

feat: reply to invalid config #852

Closed gitcoindev closed 12 months ago

gitcoindev commented 12 months ago

The bot replies directly to invocations with invalid config to make it clear why it's down. A new helper method upsertLastCommentToIssue was introduce to avoid adding multpile comments with the same error message.

Resolves: #799

Quality Assurance: https://github.com/gitcoindev/ubiquibot/issues/12

netlify[bot] commented 12 months ago

Deploy Preview for ubiquibot-staging ready!

Name Link
Latest commit 3a073ce72b42e949d734d8412dc912854a9bc193
Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/65268a6eaa09e6000886931c
Deploy Preview https://deploy-preview-852--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.

gitcoindev commented 12 months ago

@0xcodercrane I applied suggested changes and did a QA on issue with 182 comments:

https://github.com/gitcoindev/ubiquibot/issues/11#issuecomment-1757486779

gitcoindev commented 12 months ago

Thank you for reviewing and merging!