Skip to content

Remove Proofing::Result#add_message#6861

Merged
zachmargolis merged 3 commits intomainfrom
margolis-remove-proofing-messages
Aug 29, 2022
Merged

Remove Proofing::Result#add_message#6861
zachmargolis merged 3 commits intomainfrom
margolis-remove-proofing-messages

Conversation

@zachmargolis
Copy link
Contributor

Why: It seems to be unused

[skip changelog]

**Why**: It seems to be unused

[skip changelog]
@zachmargolis zachmargolis requested review from a team August 29, 2022 17:52
Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@theabrad theabrad left a comment

Choose a reason for hiding this comment

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

LGTM

@zachmargolis zachmargolis merged commit e04df2c into main Aug 29, 2022
@zachmargolis zachmargolis deleted the margolis-remove-proofing-messages branch August 29, 2022 19:29
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