Skip to content

Update README.md with corrected example#19

Merged
sumitmsft merged 3 commits intomicrosoft:alphafrom
pawarbi:alpha
Apr 2, 2025
Merged

Update README.md with corrected example#19
sumitmsft merged 3 commits intomicrosoft:alphafrom
pawarbi:alpha

Conversation

@pawarbi
Copy link
Contributor

@pawarbi pawarbi commented Mar 11, 2025

connection fixed:
connection_string = ("SERVER=<your_server_name>;DATABASE=<your_database_name>;UID=<your_user_name>;PWD=<your_password>;Encrypt=yes;") connection = mssql_python.connect(connection_string)

connection fixed:
connection_string = ("SERVER=<your_server_name>;DATABASE=<your_database_name>;UID=<your_user_name>;PWD=<your_password>;Encrypt=yes;")
connection = mssql_python.connect(connection_string)
@bayooyetoro
Copy link

For some reasons, this doesn't work yet;
"module 'mssql_python' has no attribute 'connect'"

@bewithgaurav
Copy link
Collaborator

@pawarbi thanks for updating this, I have committed a minor change to import connect() method directly
Requesting you to fix the merge conflicts before we can merge
@sumitmsft - Please review

@bewithgaurav
Copy link
Collaborator

For some reasons, this doesn't work yet; "module 'mssql_python' has no attribute 'connect'"

@bayooyetoro - the driver just supports python 3.13 as of now, could you please check your python version and test on python 3.13?

@sumitmsft sumitmsft merged commit d867502 into microsoft:alpha Apr 2, 2025
1 check passed
Copy link
Contributor Author

@pawarbi pawarbi left a comment

Choose a reason for hiding this comment

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

looks good

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.

4 participants