STYLE: Adding precommit hook and workflow for checking code quality

[jahnvi480]

Work Item / Issue Reference

AB#33454

---

Summary

This pull request introduces automated code quality checks for both Python and C++ files, ensuring consistent linting standards across the project. It adds configuration files and workflow automation to enforce linting thresholds, error limits, and style guidelines. The most important changes are grouped below by theme.

Continuous Integration and Workflow Automation:

• Added .github/workflows/code-quality-check.yml to run linting checks automatically on pull requests for Python and C++ files, including status reporting and PR comments.

Linting Configuration:

• Created .pre-commit-config.yml to enable pre-commit hooks for pylint and cpplint, ensuring code is linted before commits are made.
• Added pyproject.toml configuration for pylint, specifying disabled checks, minimum score threshold, and maximum line length.
• Added cpplint.cfg to configure C++ linting rules, such as excluding build files, setting line length, and disabling specific readability checks.

[Copilot]

Pull Request Overview

This pull request adds automated code quality enforcement by introducing pre-commit hooks and GitHub Actions workflows to check Python and C++ code against linting standards.

Key changes:

• GitHub Actions workflow for automatic linting checks on pull requests with configurable thresholds
• Pre-commit hooks for local linting enforcement before commits
• Configuration files defining linting rules, disabled checks, and error thresholds for both Python (pylint) and C++ (cpplint)

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

File	Description
.github/workflows/code-quality-check.yml	Implements CI workflow to run pylint and cpplint on PRs, report status, and post results as comments
.pre-commit-config.yml	Configures pre-commit hooks for pylint and cpplint with specified arguments and file exclusions
pyproject.toml	Defines pylint configuration including disabled checks, minimum score threshold of 8.5, and max line length
cpplint.cfg	Sets C++ linting rules including line length limit and filtered checks

---

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

[github-actions]

📊 Code Coverage Report

🔥 Diff Coverage 100%

🎯 Overall Coverage 77%

📈 Total Lines Covered: 4822 out of 6206 📁 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.helpers.py: 67.8%
mssql_python.pybind.ddbc_bindings.cpp: 70.7%
mssql_python.pybind.connection.connection.cpp: 74.4%
mssql_python.pybind.connection.connection_pool.cpp: 78.9%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.pybind.ddbc_bindings.h: 79.7%
mssql_python.auth.py: 87.1%
mssql_python.pooling.py: 87.7%
mssql_python.__init__.py: 90.9%
mssql_python.exceptions.py: 92.8%

---

🔗 Quick Links

⚙️ Build Summary	📋 Coverage Details
View Azure DevOps Build	Browse Full Coverage Report

[sumitmsft]

only minor change required. rest all looks good