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

Convert XLS files to XLSX to be compatible with xl2times #64

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

siddharth-krishna
Copy link
Collaborator

@siddharth-krishna siddharth-krishna commented Dec 18, 2024

xl2times does not support the old XLS file format, so this PR converts the 2 files in the old format to the newer XLSX format.

I don't have Excel, so I did this using an online tool. Once #63 is merged in, we can use raw_tables.txt to verify that the changes in this PR did not change the contents of the two files. EDIT: alas, we can't use it because the old XLS files were not included in raw_tables.txt (doh!). Is there some way to check that the new files are equivalent to the old ones?

Copy link
Collaborator

@willcattoneeca willcattoneeca left a comment

Choose a reason for hiding this comment

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

Awesome, thanks Sid!

Copy link
Collaborator

@willcattoneeca willcattoneeca left a comment

Choose a reason for hiding this comment

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

Awesome, thanks Sid!

@siddharth-krishna
Copy link
Collaborator Author

Thanks Will. I can push the new raw tables file in a few minutes to make this pass CI.

@willcattoneeca
Copy link
Collaborator

Thanks Will. I can push the new raw tables file in a few minutes to make this pass CI.

Ah, have already done this!

@searleluke searleluke merged commit fce02b0 into main Dec 19, 2024
1 check passed
@searleluke searleluke deleted the sid/xls-to-xlsx branch December 19, 2024 03:53
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.

3 participants