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

Add pdf2image, Spire.Doc and Spire.Xls as packages #399

Closed
wants to merge 3 commits into from
Closed

Add pdf2image, Spire.Doc and Spire.Xls as packages #399

wants to merge 3 commits into from

Conversation

D-Abdi
Copy link

@D-Abdi D-Abdi commented Mar 15, 2024

Hi,

I requested a few packages that I have had some trouble with getting it to work in my Lambda function. If you could add support for these that would be great!

Ps: Love your work!

@keithrozario
Copy link
Owner

Thanks for contributing.

But the format of the excel is wrong, each Python package should be on it's own line of the CSV. i.e. only one package per line, if you wish to add more than one package -- you can add more lines.

Also I'm not sure what spire.doc or spire.xls are -- are they python packages available on PyPi?

@D-Abdi
Copy link
Author

D-Abdi commented Mar 20, 2024

Thanks for contributing.

But the format of the excel is wrong, each Python package should be on it's own line of the CSV. i.e. only one package per line, if you wish to add more than one package -- you can add more lines.

Also I'm not sure what spire.doc or spire.xls are -- are they python packages available on PyPi?

Hi,

Thank you for the heads-up. I have mutated the CSV to have each package on a separate line.
Also, yes spire.doc and spire.xls are python packages available PyPi that are used to accomplish a similar goal to pdf2image but for the files with a doc, docx, xls and xlsx format.

@keithrozario
Copy link
Owner

almost there ... each line is 3 columns (so we need 3 commas), and the email is the contact of the author.

@keithrozario
Copy link
Owner

sorry to be pedantic, but if the format of the file is wrong, the entire pipeline might fail :(

@D-Abdi D-Abdi closed this May 22, 2024
@D-Abdi D-Abdi deleted the d-abdi-master branch May 22, 2024 07:57
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.

2 participants