Skip to content

Feature/sql server - #119

Merged
rabindra-neupane-intugle merged 4 commits into
mainfrom
feature/sql-server
Nov 10, 2025
Merged

Feature/sql server#119
rabindra-neupane-intugle merged 4 commits into
mainfrom
feature/sql-server

Conversation

@raphael-intugle

@raphael-intugle raphael-intugle commented Nov 7, 2025

Copy link
Copy Markdown
Collaborator

Description

Added SQL Server connector

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🎨 Code style update (formatting, renaming)
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Test update
  • 🔧 Configuration change
  • 🏗️ Infrastructure/build change

Related Issue(s)

#114

Changes Made

  • Uses the latest mssql-python library

Testing

Test Configuration

  • Python Version:
  • OS:
  • LLM Provider (if applicable):

Test Cases

  • Unit tests pass locally
  • Manual testing completed
  • Tested with sample datasets

Test Commands

# Commands to run tests
pytest tests/

Screenshots/Examples

# Example usage of new feature/fix

Checklist

  • My code follows the code style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or linter errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published
  • I have updated the relevant notebooks (if applicable)
  • I have checked my code and corrected any misspellings

Documentation Updates

  • README.md updated
  • Docstrings added/updated
  • Documentation site updated (if needed)
  • Notebook examples updated (if applicable)
  • CHANGELOG updated (if applicable)

shiva-intugle
shiva-intugle previously approved these changes Nov 7, 2025
@raphael-intugle
raphael-intugle dismissed shiva-intugle’s stale review November 7, 2025 06:11

The merge-base changed after approval.

shiva-intugle
shiva-intugle previously approved these changes Nov 7, 2025
@raphael-intugle
raphael-intugle dismissed shiva-intugle’s stale review November 7, 2025 06:17

The merge-base changed after approval.

@shiva-intugle
shiva-intugle self-requested a review November 7, 2025 06:17
shiva-intugle
shiva-intugle previously approved these changes Nov 7, 2025
@raphael-intugle
raphael-intugle dismissed shiva-intugle’s stale review November 7, 2025 06:33

The merge-base changed after approval.

@raphael-intugle
raphael-intugle dismissed narotsit-intugle’s stale review November 7, 2025 06:55

The merge-base changed after approval.

@raphael-intugle
raphael-intugle dismissed juhel-phanju-intugle’s stale review November 7, 2025 07:29

The merge-base changed after approval.

@rabindra-neupane-intugle rabindra-neupane-intugle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok to merge

@raphael-intugle
raphael-intugle dismissed rabindra-neupane-intugle’s stale review November 7, 2025 07:35

The merge-base changed after approval.

prinkanintugle
prinkanintugle previously approved these changes Nov 8, 2025
@rabindra-neupane-intugle
rabindra-neupane-intugle merged commit 60826be into main Nov 10, 2025
5 checks passed
@rabindra-neupane-intugle
rabindra-neupane-intugle deleted the feature/sql-server branch November 10, 2025 10:17
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