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

fix(client): Support playbooks with unicode for Python 3.12+ #4342

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

pkoprda
Copy link
Contributor

@pkoprda pkoprda commented Jan 20, 2025

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • No Sensitive Data in this change?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Complete Description of Additions/Changes:

  • Card ID: CCT-644

Some Unicode characters, like the zero-width joiner in emojis, were not processed correctly. This update improves Unicode support, mainly aimed for Python 3.12+.

This pull request is a backport of: insights-ansible-playbook-verifier#32.

@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.06%. Comparing base (f28e238) to head (05e10ce).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4342   +/-   ##
=======================================
  Coverage   77.06%   77.06%           
=======================================
  Files         736      736           
  Lines       41368    41368           
  Branches     8789     8789           
=======================================
+ Hits        31879    31881    +2     
+ Misses       8429     8428    -1     
+ Partials     1060     1059    -1     
Flag Coverage Δ
unittests 77.05% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@xiangce xiangce added the client These issues represent work to be done by the "client" team. label Jan 27, 2025
@pkoprda pkoprda force-pushed the unicode branch 4 times, most recently from 6e76db9 to 9ea9ed2 Compare January 28, 2025 16:19
* Card ID: CCT-644

Some Unicode characters, like the zero-width joiner in emojis, were not
processed correctly. This update improves Unicode support, mainly aimed
for Python 3.12+.

Signed-off-by: pkoprda <pkoprda@redhat.com>
Copy link
Contributor

@m-horky m-horky left a comment

Choose a reason for hiding this comment

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

LGTM. @xiangce when you are ready :)

@xiangce xiangce merged commit c446793 into RedHatInsights:master Feb 5, 2025
11 of 14 checks passed
@pkoprda pkoprda deleted the unicode branch February 5, 2025 08:51
xiangce pushed a commit that referenced this pull request Feb 6, 2025
* Card ID: CCT-644

Some Unicode characters, like the zero-width joiner in emojis, were not
processed correctly. This update improves Unicode support, mainly aimed
for Python 3.12+.

Signed-off-by: pkoprda <pkoprda@redhat.com>
(cherry picked from commit c446793)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client These issues represent work to be done by the "client" team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants