-
Notifications
You must be signed in to change notification settings - Fork 1
Description
This serves as a PR comment template for the new CI scripts.
All good
Hello @padarom and thank you for your contribution. We have run some automated tests to make sure that your pull request matches our contribution gudelines.
🎉 We have not found any issues with your pull request, so it is now good to go. You can now lean back and wait for a review, after which your app will be released.
If you make any changes to your pull request after this comment, the automated tests will run again and this comment will be updated accordingly. If any errors pop up then, please come back to this comment to find out what went wrong.
Some errors present
Hello @padarom and thank you for your contribution. We have run some automated tests to make sure that your pull request matches our contribution gudelines.
- Your new app must live in a directory with the same name. If your app is named
parrots-for-president, it must be in/apps/parrots-for-president. - TBD
Please try to fix these issues, as otherwise we cannot guarantee that we will be able to review and accept your contribution. If you have any questions, feel free to ask them in the official Awtrix Discord.
Once you make any changes to your pull request, the automated tests will run again and this comment will be updated accordingly. If any errors pop up then, please come back to this comment to find out what went wrong.