Conversation
a151a0d
to
33191e6
Compare
33191e6
to
74f9011
Compare
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.
This LGTM! It seems reasonable enough to include, so we'll have it as part of the next release. Thanks!
@zkat Just a heads up! I originally wrote this PR against I'm afraid it might need to be rewritten 😢 |
@tribou I think you'll find the conflict to be very small! It would be great if you just resolved the conflict and pushed out a new, rebased version -- or I can do it myself when it's time to merge (although that'd be more work for me lolsob). Your patch is straightforward enough :) |
Ah, I didn't realize that! I'll try to look into it soon |
I'm doing my merges today, and it looks like this isn't ready yet, so I'm pushing it out until next time we do a PR sweep. |
83a5d44
to
a571f80
Compare
bcc4670
to
4a65b3b
Compare
2e76172
to
fe31e66
Compare
63c9452
to
1b021d0
Compare
691cb35
to
bf3cfa7
Compare
3f32220
to
f29b7a6
Compare
Adds
sign-git-commit
config referenced in #12696