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

anonymize_list return type hint fix #1178

Merged
merged 4 commits into from
Sep 19, 2023
Merged

anonymize_list return type hint fix #1178

merged 4 commits into from
Sep 19, 2023

Conversation

bl4ckwind
Copy link
Contributor

Change Description

The anonymize_list function of the BatchAnonymizerEngine class never returns List[EngineResult].
Changed the return type hint to List[Union[str, object]] and import

Checklist

  • I have reviewed the contribution guidelines
  • I have signed the CLA (if required)
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

@omri374
Copy link
Contributor

omri374 commented Sep 19, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@omri374 omri374 merged commit f36caf4 into microsoft:main Sep 19, 2023
@omri374
Copy link
Contributor

omri374 commented Sep 19, 2023

Thanks! merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants