Skip to content

feat: refactor emojify modules#322

Merged
wax911 merged 9 commits into
developfrom
feat/226-refactor-emojify-modules
Mar 1, 2025
Merged

feat: refactor emojify modules#322
wax911 merged 9 commits into
developfrom
feat/226-refactor-emojify-modules

Conversation

@wax911

@wax911 wax911 commented Feb 9, 2025

Copy link
Copy Markdown
Member

AniTrend Pull Request

Thank you for contributing! Please take a moment to review our contributing guidelines to make the process easy and effective for everyone involved.

Please open an issue before embarking on any significant pull request, especially those that add a new library or change existing tests, otherwise you risk spending a lot of time working on something that might not end up being merged into the project.

Before opening a pull request, please ensure you've done the following:

  • You have followed our contributing guidelines
  • Double checked that your branch is based on develop and targets develop (where applicable)
  • Pull request has tests (if applicable)
  • Documentation is updated (if necessary)
  • Description explains the issue/use-case resolved

Description

Closes #226

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (breaking change which adds functionality)
  • Enhancement (Improves existing functionality)

IMPORTANT: By submitting a patch, you agree to allow the project owners to license your work under the terms of the Apache License 2.0.

@wax911 wax911 changed the base branch from develop to feat/226-platform-actions-create-automated-action-to-generate-emojijson-inline-with-unicodeorg February 9, 2025 20:30
@wax911 wax911 added the breaking label Feb 9, 2025
@wax911 wax911 force-pushed the feat/226-platform-actions-create-automated-action-to-generate-emojijson-inline-with-unicodeorg branch from 1dc9f48 to b25e7bb Compare February 23, 2025 11:08
Base automatically changed from feat/226-platform-actions-create-automated-action-to-generate-emojijson-inline-with-unicodeorg to develop February 23, 2025 11:21
@wax911 wax911 force-pushed the feat/226-refactor-emojify-modules branch 2 times, most recently from ee7aaed to 04739b4 Compare February 23, 2025 13:06
@wax911 wax911 force-pushed the feat/226-refactor-emojify-modules branch from 04739b4 to 100d70d Compare March 1, 2025 11:11
wax911 added 4 commits March 1, 2025 13:32
…test case

- Renamed `AbstractEmojiExtension` to `EmojiExtension`.
- Updated `EmojiParseTest` to reflect the new name.
- Adjusted the expected emoji in `removeEmojis_only_removes_the_emojis_in_the_iterable_from_the_string` to not have skin tone.
- Removes xml declaration from `misc.xml`.
- Updates module file paths in `modules.xml`.
- Updates app id value in `appInsightsSettings.xml`.
@wax911 wax911 force-pushed the feat/226-refactor-emojify-modules branch from ae39c8f to 9147f6c Compare March 1, 2025 12:14

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Auto approved automated PR

@wax911 wax911 merged commit e994f22 into develop Mar 1, 2025
@wax911 wax911 deleted the feat/226-refactor-emojify-modules branch March 1, 2025 12:19
@wax911 wax911 added the feature label Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

platform(actions): create automated action to generate emoji.json inline with unicode.org

1 participant