chris48s
chris48s
TODO: use a PAT if the PR is from a local branch (like the integration tests)
@calebcartwright - this one is now ready for a review
It has been 6 months since I last worked on this branch, so it is not really fresh in my mind, but I will try and find some time to...
https://github.com/badges/shields/pull/10320 adds a daily tests repo back to the repo. I would still like to add coverage reports to that though.
Don't worry about squashing or rebasing. We squash-merge PRs on this project anyway. Just keep pushing more commits to the branch. It is easier to see what has changed between...
I feel like this PR adds a couple of features that are nearly ready to go but we're also going round in circles on some other things. I'm going to...
You already did this: https://github.com/badges/shields/pull/2890 but its disabled in production because if a user tries to load it, it immediately bans their IP Probably the job here is to see...
I think I'm pretty happy with this way this is currently handled (with links out to https://simpleicons.org/ and https://github.com/simple-icons/simple-icons/blob/master/slugs.md ). We could still consider a picker. The full SimpleIcons set...
While I agree with the general thrust of this (bump comments are unhelpful - we'll update if there's any progress), I think this needs clarifying: There is already an open...
> we may have to agree to disagree on this one We don't disagree, but IMO it needed clarifying what we would actually need help with to move this (and...