Skip to content
This repository was archived by the owner on Dec 22, 2023. It is now read-only.

Added a file named mysql-python.py in MysqlPythonBasic folder, which … #559

Closed
wants to merge 1 commit into from

Conversation

Rohitm619
Copy link

@Rohitm619 Rohitm619 commented Oct 17, 2020

…will help to store the scrapped data in the database, the file is explaning the connection of mysql and python, creating database, creating table, inserting values in tables

Description

A short summary of what is included in your Pull Request.

Fixes #(issue_no)

Replace issue_no in the above line, with the issue related to this PR.

Type of change

Choosing one or more options from the following as per the nature of your Pull request.

  • NOTE: These boxes can be checked using [X]

  • Bug fix (non-breaking change which fixes an issue)

  • New feature (non-breaking change which adds functionality)

  • Documentation Update

Checklist:

Please tick all the boxes that are fulfilled by your Pull Request.

  • I have named my files and folder, according to this project's guidelines.
  • My code follows the style guidelines of this project.
  • My Pull Request has a descriptive title. (not a vague title like Update index.md)
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
  • I have included a requirements.txt file (if external libraries are required.)
  • My changes do not produce any warnings.
  • I have added a working sample/screenshot of the script.

…will help to store the scrapped data in the database, the file is explaning the connection of mysql and python, creating database, creating table, inserting values in tables

print(name)

if name!=None or gender!=None:

Choose a reason for hiding this comment

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

The Deepsource test is failed due to

if name is not None or gender is not None:

Comparisons to the singleton objects, like True, False, and None, should be done with identity, not equality. Use “is” or “is not”. Identity checks are faster than equality checks.

import mysql.connector
import pyautogui

mypasswd=pyautogui.prompt('Enter your mysql password')

Choose a reason for hiding this comment

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

use whitespace around the operator

@AdityaJ7
Copy link
Member

Closing this, for now, please follow the usual drill - you have to create an issue before sending a PR, and also stick to the repo guidelines about Readme files and stuff that should be present in your script folder.

Thanks

@AdityaJ7 AdityaJ7 closed this Oct 18, 2020
@github-actions
Copy link

Python World does not accept pull requests on this repository anymore.

@github-actions github-actions bot locked and limited conversation to collaborators Jun 10, 2022
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