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

Spreadsheet / XLSX parser and writer #138

Closed
wants to merge 6 commits into from
Closed

Spreadsheet / XLSX parser and writer #138

wants to merge 6 commits into from

Conversation

ianling
Copy link
Collaborator

@ianling ianling commented Apr 18, 2022

Adds a spreadsheet/XLSX/XLS parser and writer. Closes #136

Includes unit tests:

  1. Parses the example XLSX file from here: https://github.com/spdx/spdx-spec/tree/development/v2.2.2/examples and then checks the result against a handwritten Document struct
  2. Writes out the handwritten struct to a XLSX file, and then parses the written XLSX file to verify that no data was lost and all the values match what was expected.

Depends on and contains commits from #133, #134, and #137

Also bumps the Github Actions workflow to use Go 1.15, because that is the minimum version supported for the Excelize module used for interacting with XLSX files.

@ianling ianling changed the title WIP: Spreadsheet / XLSX parser and writer Spreadsheet / XLSX parser and writer Apr 20, 2022
@ianling ianling marked this pull request as ready for review April 20, 2022 19:00
Signed-off-by: Ian Ling <ian@iancaling.com>
Signed-off-by: Ian Ling <ian@iancaling.com>
Signed-off-by: Ian Ling <ian@iancaling.com>
Signed-off-by: Ian Ling <ian@iancaling.com>
Signed-off-by: Ian Ling <ian@iancaling.com>
Signed-off-by: Ian Ling <ian@iancaling.com>
@kzantow
Copy link
Collaborator

kzantow commented Jan 18, 2023

Hi again @ianling -- are you able to adjust this PR for the recent refactoring?

@kzantow
Copy link
Collaborator

kzantow commented May 22, 2023

Hiya @ianling -- thank you for this (and the other) contributions. I'm also going to close this for now, since it's been a few months open without movement. We will gladly reopen this if you decide to update/continue it!

@kzantow kzantow closed this May 22, 2023
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.

XLSX / Spreadsheet Parser and Writer
2 participants