Skip to content

Rework blame exceptions #53

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

Closed
wants to merge 1 commit into from
Closed

Conversation

real-or-random
Copy link
Collaborator

@real-or-random real-or-random commented Nov 27, 2024

Collecting some early feedback here.

@real-or-random real-or-random marked this pull request as draft November 27, 2024 10:46
@real-or-random real-or-random force-pushed the 202411-blame-exceptions branch 2 times, most recently from 4fe2f95 to 5fe9298 Compare December 4, 2024 14:26
@real-or-random real-or-random changed the title wip rework blame exceptions Rework blame exceptions Dec 4, 2024
@real-or-random real-or-random marked this pull request as ready for review December 4, 2024 14:27
@real-or-random real-or-random force-pushed the 202411-blame-exceptions branch from 5fe9298 to 81a9a24 Compare December 4, 2024 14:29
@real-or-random
Copy link
Collaborator Author

Ready for review.

Meanwhile, I'll continue with the docs of the blame functions.

@real-or-random real-or-random force-pushed the 202411-blame-exceptions branch 4 times, most recently from dcfc595 to d30aa56 Compare December 4, 2024 15:07
@real-or-random real-or-random force-pushed the 202411-blame-exceptions branch from d30aa56 to 513e340 Compare December 4, 2024 15:08
@real-or-random
Copy link
Collaborator Author

I accidentally pushed most of this to master already in f3a65a0. Please review there.

Copy link
Collaborator

@jonasnick jonasnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

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.

2 participants