python-zulip-api icon indicating copy to clipboard operation
python-zulip-api copied to clipboard

Published 20 hours ago • verified publisher icon zulip

integrations: Remove !avatar from git and game handler.

Open ahmedabuamra opened this issue 4 years ago • 5 comments

Fixes: #632

Message content example (on my forked repo.): image

Please note this sample is generated with input: 5f81eb3a7071fa0dfc20335d01ad889edca27f4e e995e52896a9fa93678eedb777d6d63927913bb5 refs/heads/master

which formatting output of: git log --reverse --pretty=%aN%n%H%n%h%n%s 5f81eb3a7071fa0dfc20335d01ad889edca27f4e..e995e52896a9fa93678eedb777d6d63927913bb5

ahmedabuamra avatar Feb 05 '21 04:02 ahmedabuamra

@andersk thank you for your comments! I agree with all of them and will update you once resolved.

ahmedabuamra avatar Feb 05 '21 17:02 ahmedabuamra

This looks pretty good; I merged the first commit as 9ce2ea53a50cf80554c9014ebfa500a8f211b529 (after updating the commit message to say Fixes part of #632 -- the way you had it would close the issue as soon as the first commit is merged, even if the second wasn't ready yet.

timabbott avatar Feb 19 '21 02:02 timabbott

@ahmedabuamra have you had a chance to resolve the feedback on this?

timabbott avatar Mar 25 '21 00:03 timabbott

@timabbott I am sorry for not working on it till now. I have had exams from the beginning of this month to yesterday. I will resolve it as soon as possible. Thank you.

ahmedabuamra avatar Mar 25 '21 08:03 ahmedabuamra

Heads up @ahmedabuamra, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/main branch and resolve your pull request's merge conflicts accordingly.

zulipbot avatar Aug 24 '21 19:08 zulipbot