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

chore: Simplify Text Analytics APIs and remove Dependence on Azure SDKs #1622

Merged
merged 2 commits into from
Aug 23, 2022

Conversation

mhamilton723
Copy link
Collaborator

@mhamilton723 mhamilton723 commented Aug 19, 2022

What changes are proposed in this pull request?

Removing Azure SDK dependencies to make SynapseML easier to install

@github-actions
Copy link

Hey @mhamilton723 👋!
Thank you so much for contributing to our repository 🙌.
Someone from SynapseML Team will be reviewing this pull request soon.
We appreciate your patience and contributions 💯!

@mhamilton723 mhamilton723 changed the title Remove dependence on Azure SDKs chore: Remove dependence on Azure SDKs Aug 19, 2022
@mhamilton723 mhamilton723 force-pushed the remove-azure-sdk branch 2 times, most recently from 22ee187 to 1a7bbc4 Compare August 23, 2022 00:48
@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2022

Codecov Report

Merging #1622 (3d0f3f4) into master (3d0f3f4) will not change coverage.
The diff coverage is n/a.

❗ Current head 3d0f3f4 differs from pull request most recent head 1582a0f. Consider uploading reports for the commit 1582a0f to get more accurate results

@@           Coverage Diff           @@
##           master    #1622   +/-   ##
=======================================
  Coverage   83.61%   83.61%           
=======================================
  Files         288      288           
  Lines       15334    15334           
  Branches      747      747           
=======================================
  Hits        12821    12821           
  Misses       2513     2513           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mhamilton723 mhamilton723 force-pushed the remove-azure-sdk branch 2 times, most recently from cb0816e to 98f59a9 Compare August 23, 2022 01:43
@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 force-pushed the remove-azure-sdk branch 2 times, most recently from d9f80bb to 542b9e5 Compare August 23, 2022 02:25
@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 changed the title chore: Remove dependence on Azure SDKs chore: Simplify Text Analytics APIs and remove Dependence on Azure SDKs Aug 23, 2022
@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit dbbe681 into microsoft:master Aug 23, 2022
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.

None yet

2 participants