Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

NewProjectVersion #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pagamatteo
Copy link

New version of the project.

@msftclas
Copy link

msftclas commented Oct 24, 2019

CLA assistant check
All CLA requirements met.

Comment on lines 13 to 15
#self.password = "ML+matteo<3paolo+SQL"
self.password = "ndulsp+92+pgnll"
dialect = 'mysql+pymysql'
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this ok? If we use this psw somewhere we will have to change it

Comment on lines 5 to 9
public static string MYSQLConnection = "server=localhost;Uid=root;Pwd=ML+matteo<3paolo+SQL;Database=MLtoSQL";
public static string SQLSERVERConnection = "server=localhost;Uid=SA;Pwd=ML+matteo<3paolo+SQL;Database=mltosql";

//public static string MYSQLConnection = "server=localhost;Uid=root;Pwd=19021990;Database=MLtoSQL";
// public static string SQLSERVERConnection = "server=localhost;Uid=root;Pwd=19021990;Database=MLtoSQL";
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these safe or we have to change the psw to something now?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants