Skip to content

FIX: install requirements.txt in Windows build stage to fix psutil import error#507

Merged
jahnvi480 merged 1 commit intomainfrom
saumya/fix-build-pipeline-windows-stage
Apr 10, 2026
Merged

FIX: install requirements.txt in Windows build stage to fix psutil import error#507
jahnvi480 merged 1 commit intomainfrom
saumya/fix-build-pipeline-windows-stage

Conversation

@gargsaumya
Copy link
Copy Markdown
Contributor

@gargsaumya gargsaumya commented Apr 10, 2026

Work Item / Issue Reference

AB#43934

GitHub Issue: #<ISSUE_NUMBER>


Summary

This pull request makes a minor update to the Windows build pipeline by adding the psutil package to the list of Python dependencies installed during the build process.

  • Build pipeline dependencies:
    • Added psutil to the pip install command in OneBranchPipelines/stages/build-windows-single-stage.yml to ensure it is available during builds.

Copilot AI review requested due to automatic review settings April 10, 2026 03:33
@github-actions github-actions bot added the pr-size: small Minimal code update label Apr 10, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Windows OneBranch build stage dependency installation to rely on the repo’s requirements.txt, aiming to resolve missing dependency issues (notably psutil) during Windows CI runs and align Windows behavior with other pipelines that already install from requirements.txt.

Changes:

  • Switch Windows dependency installation from an explicit package list to pip install -r requirements.txt plus pyodbc.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gargsaumya gargsaumya force-pushed the saumya/fix-build-pipeline-windows-stage branch from ee65b18 to 8e90b03 Compare April 10, 2026 03:37
@github-actions
Copy link
Copy Markdown

📊 Code Coverage Report

🔥 Diff Coverage

100%


🎯 Overall Coverage

79%


📈 Total Lines Covered: 6639 out of 8403
📁 Project: mssql-python


Diff Coverage

Diff: main...HEAD, staged and unstaged changes

No lines with coverage information in this diff.


📋 Files Needing Attention

📉 Files with overall lowest coverage (click to expand)
mssql_python.pybind.logger_bridge.cpp: 59.2%
mssql_python.pybind.ddbc_bindings.h: 67.8%
mssql_python.row.py: 70.5%
mssql_python.pybind.logger_bridge.hpp: 70.8%
mssql_python.pybind.ddbc_bindings.cpp: 74.2%
mssql_python.pybind.connection.connection.cpp: 75.3%
mssql_python.__init__.py: 77.3%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.pybind.connection.connection_pool.cpp: 79.6%
mssql_python.connection.py: 85.2%

🔗 Quick Links

⚙️ Build Summary 📋 Coverage Details

View Azure DevOps Build

Browse Full Coverage Report

@jahnvi480 jahnvi480 merged commit 2b6c5f9 into main Apr 10, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-size: small Minimal code update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants