Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Fix traceback on Abuse Report." #293

Merged
merged 1 commit into from
May 13, 2022
Merged

Conversation

Yoric
Copy link
Contributor

@Yoric Yoric commented May 13, 2022

@Yoric Yoric merged commit bcc3405 into main May 13, 2022
@Yoric Yoric deleted the revert-230-fix-handlereport-cb branch May 13, 2022 13:29
@maranda
Copy link
Contributor

maranda commented May 13, 2022

@Yoric I'm not sure I follow where it breaks stuff... https://github.com/matrix-org/mjolnir/runs/6423584294?check_suite_focus=true

@maranda
Copy link
Contributor

maranda commented May 13, 2022

Also why are you referencing my own CI that usually has several issues instead of matrix-org's?

@Yoric
Copy link
Contributor Author

Yoric commented May 13, 2022

I actually have no idea what happened. I don't think the problem is in your code, but when I merged it, something broke. I've reverted just in case, to be able to take a second look without blocking other devs.

@Gnuxie
Copy link
Contributor

Gnuxie commented May 17, 2022

What broke? seemed fine 27ae3ec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants