-
Notifications
You must be signed in to change notification settings - Fork 61
Conversation
✅ Deploy Preview for ubiquibot-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good to me. 👍
This will help new contributors who having issues running the bot.
@ubiquity/software-development Please take a look
everyone check QA https://github.com/molecula451/ubiquibot/blob/update-readme/README.md |
it looks like pavlovcik wants to also integrate the supabase migration thing #838 (comment) but! the supabase readme it's complete and i think a dev should be aware of this module at the repo and how to handle the inner stuff of supabase, e.g migrations what do you guys think? @rndquu @0xcodercrane @wannacfuture @whilefoo |
I skimmed this. It looks really substantial! Good work! I do not have time to test the set up right now but I expect that the other team members should try from A to Z to set up a new bot and follow that tutorial. |
I have tried this readme(setting up the bot) and looks good to me. 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve based on @wannacfuture's review. However it would be appreciated if we could get at least one more before merging.
yeah i agree @rndquu @0xcodercrane |
@0xcodercrane crane let's get an eagle eye on this one to help pavlovcik get it merge |
@whilefoo when you approve this will automatically merge. Thanks everybody for your reviews! |
@whilefoo your review it's important for everyone here |
Resolves #838
Quality Assurance: https://github.com/molecula451/ubiquibot/blob/update-readme/README.md