Skip to content

chore: dev to main merge#59

Closed
Saswato-Microsoft wants to merge 9 commits into
mainfrom
dev
Closed

chore: dev to main merge#59
Saswato-Microsoft wants to merge 9 commits into
mainfrom
dev

Conversation

@Saswato-Microsoft

Copy link
Copy Markdown
Collaborator

Purpose

This pull request primarily focuses on cleaning up unused imports across several Python infrastructure scripts, improving code clarity and maintainability. Additionally, it introduces a version exclusion for the azd tool in the Azure configuration.

Code cleanup and import optimization:

  • Removed unused imports such as argparse, sys, and various typing hints (Dict, Any, Optional, List) from multiple files including fabric_common_utils.py, fabric_database.py, fabric_eventhouse.py, fabric_eventhub.py, fabric_eventstream_definition.py, fabric_workspace.py, and graph_api.py to streamline the codebase. [1] [2] [3] [4] [5] [6] [7] [8]

Dependency management:

  • Updated the azure.yaml configuration to exclude version 1.23.9 of the azd tool, likely due to compatibility or stability concerns.

Dependency usage:

  • Removed the unused ClientRequestProperties import from azure.kusto.data in fabric_database.py.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Saswato-Microsoft

Copy link
Copy Markdown
Collaborator Author

Creating a new updated pull request, hence closing this pull request as of now.

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.

6 participants