Closed banjo-tech closed 2 years ago
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help.
Dangerbot
will test out your code and reply in a bit with some pointers and requests.
Also please refer here for installation help 💿
There may be some errors, but don't worry! We'll work through them with you! 👍🎉😄
One thing that can help to get started is to make sure you've included a link back to the original issue you're solving, in the format fixes #0000
(for example). And to make sure the PR title describes what you're trying to do! (often it can be the same as the issue title) Thanks! 🙌
Then, you can say hello in our chatroom & share a link to this PR to get a review! 👋 ✅
Code Climate has analyzed commit b4dfaeb5 and detected 0 issues on this pull request.
View more on Code Climate.
Merging #11514 (12285cf) into main (a5d2483) will decrease coverage by
81.49%
. The diff coverage isn/a
.:exclamation: Current head 12285cf differs from pull request most recent head b4dfaeb. Consider uploading reports for the commit b4dfaeb to get more accurate results
Hi @banjo-tech, can you add a link to the issue?
One thing that can help to get started is to make sure you've included a link back to the original issue you're solving, in the format fixes #0000 (for example)
Fixes #0000
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment below