Skip to content

fix: updated the documents and resolve the code quality issue - #1109

Merged
Roopan-Microsoft merged 1 commit into
devfrom
psl-v5pipelinefix
Jul 21, 2026
Merged

fix: updated the documents and resolve the code quality issue#1109
Roopan-Microsoft merged 1 commit into
devfrom
psl-v5pipelinefix

Conversation

@Dhruvkumar-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request primarily contains documentation cleanup and minor code refactoring. The most significant changes are the removal of outdated or unnecessary documentation files, the addition and clarification of deployment parameter documentation, and small code cleanup in test and script files.

Documentation updates and cleanup:

  • Removed the docs/ManualAzureDeployment.md guide, which described manual Azure deployment steps, as it is likely outdated or superseded.
  • Removed the docs/feature-changelog.md file that tracked high-level features for a past release.
  • Added documentation for the AZURE_ENV_DEPLOYMENT_FLAVOR parameter and clarified the table of environment variables in docs/CustomizingAzdParameters.md.
  • Minor formatting fix in the environment variables table in docs/CustomizingAzdParameters.md.

Code cleanup:

  • Removed an unused json import from infra/scripts/post-provision/upload_team_config.py.
  • Cleaned up unused imports in src/tests/backend/common/utils/test_team_utils.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]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL382455285% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
834 0 💤 0 ❌ 0 🔥 9.995s ⏱️

@Roopan-Microsoft
Roopan-Microsoft merged commit d478cd6 into dev Jul 21, 2026
8 checks passed
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