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

Chatter v1.2.1.0 #4495

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Chatter v1.2.1.0 #4495

merged 2 commits into from
Aug 29, 2024

Conversation

jlkeesey
Copy link
Contributor

Added plural handling for messages.

Moved more messages to the message catalog including some that require pluralization.

  • Various cleanups and conversion to ImRaii.

Before I knew that ImRaii existed, I created a similar library, so converting wasn't very difficult. Using theirs because it's supported and handles more cases than mine and why have duplicate code?

 Added plural handling for messages.

Moved more messages to the message catalog including some that require pluralization.

* Various cleanups and conversion to ImRaii.

Before I knew that ImRaii existed, I created a similar library, so converting wasn't very difficult. Using theirs because it's supported and handles more cases than mine and why have duplicate code?
Added plural handling for messages.
@bleatbot bleatbot enabled auto-merge (squash) August 29, 2024 05:37
@bleatbot
Copy link
Collaborator

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

The average merge time for plugin updates is currently 11 hours.

✔️Chatter [stable] - 294cdcd - v1.2.1.0 - Diff (1105 lines, prev. 1.2.0.0) - Semantic

Show log - Review

@bleatbot bleatbot added the size-large Diff for this PR is large. label Aug 29, 2024
@philpax philpax changed the title Added plural handling for messages. Chatter v1.2.1.0 Aug 29, 2024
@bleatbot bleatbot merged commit 4409e9d into goatcorp:main Aug 29, 2024
2 checks passed
@philpax
Copy link
Contributor

philpax commented Aug 29, 2024

Please ensure the name of your plugin is in your PR title 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants