Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Your data is accessible in excel file and a readme document in Turkish has been added #16

Merged
merged 5 commits into from
Apr 18, 2023

Conversation

BerkeGokturk71
Copy link
Contributor

You can See your data in Excel format and Turkish language support added.Chrome driver updated for new chrome versions.
"-e" parameter create your excel file.Now Readme File support Turkish language.I added samples.Finally I did some code reviews.

Copy link
Owner

@jasperan jasperan left a comment

Choose a reason for hiding this comment

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

Please, remove from your local (or empty) the following pull request, or add them to your .gitignore so they are not included in the PR:

  • database/database.db
  • chromedriver.exe

Also, remove .pyc files from the commit so we don't include compiled code that won't serve to other users!

@BerkeGokturk71
Copy link
Contributor Author

I added database folder and excel file and deleted the pyc folder from there.

Copy link
Owner

@jasperan jasperan left a comment

Choose a reason for hiding this comment

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

Yes, please also add chromedriver.exe to your .gitignore and remove from the main directory as well so we don't force your binary to appear in the repo!

@BerkeGokturk71
Copy link
Contributor Author

Okay I deleted chromedriver.exe . Anything else because I'm junior python developer :)

@jasperan
Copy link
Owner

jasperan commented Apr 18, 2023

congrats @BerkeGokturk71 this is a good Pull Request!

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.

None yet

2 participants