Skip to content

csv_to_excel #1

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Oct 5, 2022
Merged

csv_to_excel #1

merged 7 commits into from
Oct 5, 2022

Conversation

ayush-1909
Copy link
Contributor

@ayush-1909 ayush-1909 commented Oct 5, 2022 •
edited
Loading

A python script to convert a file from csv to excel.

@mpotane
Copy link
Owner

mpotane commented Oct 5, 2022

A python script to convert a file from csv to excel.

Have you signed up to hacktoberfest 2022 bro? Hacktoberfest2022

@mpotane mpotane linked an issue Oct 5, 2022 that may be closed by this pull request
Add csv to excel converter 🤖. #2
Closed
mpotane
mpotane reviewed Oct 5, 2022
View reviewed changes
Copy link
Owner

@mpotane mpotane left a comment

Choose a reason for hiding this comment

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

Please add requirements.txt bro. Inside it is the module openpyxl

@ayush-1909
Copy link
Contributor Author

A python script to convert a file from csv to excel.

Have you signed up to hacktoberfest 2022 bro? Hacktoberfest2022

Yes I did! This repo is eligible right?

@ayush-1909
Copy link
Contributor Author

Please add requirements.txt bro. Inside it is the module openpyxl

Done, kindly have a look

ayush-1909
ayush-1909 commented Oct 5, 2022
View reviewed changes
Copy link
Contributor Author

@ayush-1909 ayush-1909 left a comment

Choose a reason for hiding this comment

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

Changed everything as per requirements.

@mpotane
Copy link
Owner

mpotane commented Oct 5, 2022

Changed everything as per requirements.

sorry bro i had to changed it to pandas since I want it to be much more beginner-friendly.

@mpotane mpotane merged commit 319a7e9 into mpotane:main Oct 5, 2022
@ayush-1909
Copy link
Contributor Author

Changed everything as per requirements.

sorry bro i had to changed it to pandas since I want it to be much more beginner-friendly.

Ah okay, no problem! Thanks a lot :)

@mpotane
Copy link
Owner

mpotane commented Oct 5, 2022 •
edited
Loading

Changed everything as per requirements.

sorry bro i had to changed it to pandas since I want it to be much more beginner-friendly.

Ah okay, no problem! Thanks a lot :)

I have opened two issues you can contribute very easily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpotane mpotane mpotane approved these changes

Assignees

@ayush-1909 ayush-1909

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add csv to excel converter 🤖.
2 participants
@ayush-1909 @mpotane