Skip to content

fix(NODE-4771): serializeFunctions breaks function names outside of basic latin #538

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
Merged
merged 1 commit into from
Dec 8, 2022
Merged

fix(NODE-4771): serializeFunctions breaks function names outside of basic latin #538

merged 1 commit into from
Dec 8, 2022

Conversation

@nbbeeken
Copy link
Contributor

@nbbeeken nbbeeken commented Dec 7, 2022

Description

What is changing?

What is the motivation for this change?

We want the bug to appear in the changelog.

Double check the following

  • Ran npm run lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@durran durran added the Team Review Needs review from team label Dec 8, 2022
@durran durran merged commit 35a9234 into main Dec 8, 2022
@durran durran deleted the NODE-4771-serializeFunctionsBug branch December 8, 2022 15:35
@github-actions github-actions bot mentioned this pull request May 7, 2024
Closed
@github-actions github-actions bot mentioned this pull request Jun 2, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@durran durran durran approved these changes

Assignees

No one assigned

Labels

Team Review Needs review from team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nbbeeken @durran