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.

Sign up for GitHub

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

Jump to bottom

Remove plenking #1998

Merged
merged 1 commit into from
Oct 4, 2024
Merged

Remove plenking #1998

merged 1 commit into from
Oct 4, 2024

Conversation

mdosch
Copy link
Contributor

@mdosch mdosch commented Oct 3, 2024

There were two cases of plenking (a space between a word and the punctuation
marks) in front of ….

  • I ran valgrind when using my new feature
    Should not be necessary as only two strings were changed (space removed).

How to test the functionality

  • If it builds it should be fine.

@mdosch
Remove plenking
50e8237 
There were two cases of plenking (a space between a word and the punctuation
marks) in front of `…`.
@jubalh jubalh added this to the next milestone Oct 4, 2024
@jubalh jubalh added the cleanup label Oct 4, 2024
@jubalh jubalh merged commit 313ab5d into profanity-im:master Oct 4, 2024
5 checks passed
@jubalh
Copy link
Member

jubalh commented Oct 4, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jubalh jubalh jubalh approved these changes

Assignees

@mdosch mdosch

Labels
cleanup
Projects
None yet
Milestone
next
Development

Successfully merging this pull request may close these issues.
2 participants
@mdosch @jubalh