-
Notifications
You must be signed in to change notification settings - Fork 2
Added readme for csvtopdf conversion #9
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
base: main
Are you sure you want to change the base?
Conversation
SAUMILDHANKAR
commented
May 24, 2025
- Added the readme file with instructions to use the files in csvtopdf folder which will convert the csv file to a pdf file.
- This will help users to generate pdf files from csv files.
- I added all the files in chatgpt and requested to create a readme file.
|
Sorry I need to cleanup the readme file so marked it draft for now. |
Files Included"— Uses pandas and pdfkit for conversion." type entries are redundent just add once under requriemts add requriments.txt for all ".py" files in projectRunning the Scripts
Notesremove "You can replace the input CSV filename in the scripts with your own." licenselink to "LICENSE.MD" |
|
Hi @drakeredwind01 I updated the readme file as suggested and added the requirements text files. I was able to generate pdf from all the methods but had to make some code changes. Should I push those code changes as well? Please see the pdf files that I got. Looking forward to your feedback. Thank you. |
|
cool. looks great! |
|
✨ Feature: Added CSV to PDF conversion scripts and documentation This commit introduces a new set of utility scripts under Key changes include:
This addition provides clear, working examples of CSV-to-PDF functionality using popular Python libraries. |
|
Hi @drakeredwind01, thanks for your review. Your previous comment about key changes included is very helpful, I should have added something similar at the top. For the next PRs I will remember to bring it up during/after the Saturday meetup. Also, I may not join the the meetup for this week and next, because I want to use the meeting time to work on my meetup presentation. |