Skip to content

[ml] Rename MLException back to MlException#28210

Merged
harneetvirk merged 2 commits intoAzure:mainfrom
kdestin:rename-MLException-to-MlException
Jan 6, 2023
Merged

[ml] Rename MLException back to MlException#28210
harneetvirk merged 2 commits intoAzure:mainfrom
kdestin:rename-MLException-to-MlException

Conversation

@kdestin
Copy link
Member

@kdestin kdestin commented Jan 6, 2023

Description

This pull request reverts the rename MlException->MLException which was introduced in #27718, in response to an API review which flagged this as a breaking change.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-ai-ml

@harneetvirk harneetvirk merged commit f20e75f into Azure:main Jan 6, 2023
kdestin added a commit to kdestin/azure-sdk-for-python that referenced this pull request Jan 6, 2023
* Rename MLException back to MlException

* Remove extraneous backslash
kdestin added a commit that referenced this pull request Jan 7, 2023
* Rename MLException back to MlException

* Remove extraneous backslash
iscai-msft added a commit that referenced this pull request Apr 26, 2023
…into version_tolerant_models

* 'main' of https://github.com/Azure/azure-sdk-for-python: (1604 commits)
  [Storage] Matching `_shared` folder contents across all packages (#28031)
  [ml] Rename MLException back to MlException (#28210)
  [EventHubs] update readme to include uamqp backcompat section (#28198)
  [Monitor][Ingestion] Adjust upload error handling (#27556)
  Add 3.11 to QA (#28169)
  p1 component deployment exisitng job definition (#28194)
  Typing for identity (#28159)
  [ServiceBus] Allow clearing of session state (#27726)
  [ML][Pipelines] Fix parallel-for validation rule: allow empty dict as loop config item (#28178)
  [ML][Pipelines] feat: enable concurrent component registration (#28118)
  [ML][Pipelines] Test: update invalid reuse test (#28180)
  [AutoRelease] t2-loadtesting-2022-12-23-48532(can only be merged by SDK owner) (#28042)
  [ML][Pipelines] Test: update unstable tests in ci (#28157)
  [ML]: Adding experimental decorator to JobDefinition (#28200)
  [Identity] Fix 'az' existence check in CliCred (#27991)
  [EventHubs] fix tracing bug (#28149)
  ShortForMigrateFrom (#28192)
  Address the API review comments (#28188)
  [SchemaRegistryAvro] update recordings (#28158)
  [EventHub] update uamqp_transport docstring (#28186)
  ...
@kdestin kdestin deleted the rename-MLException-to-MlException branch August 10, 2023 02:15
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.

4 participants