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

Improve XLSX creation #206 #207

Merged
merged 6 commits into from
Jun 22, 2021
Merged

Improve XLSX creation #206 #207

merged 6 commits into from
Jun 22, 2021

Conversation

pombredanne
Copy link
Contributor

XlsxWriter
We now check that the content is correctly added before calling
XlsxWriter and report and error if the truncated can be truncated.
Several other adaptations of a cell content are done:

  • Create a combined license expression for expressions.
  • Keep only unique values in lists, preserving ordering.
  • Collect the main attribute of well known scan fields
  • Convert mappings to YAML.
  • Truncate the "description" field to the first five lines.
  • Convert anything unknown to string.
  • Normalize line endings
  • Truncate to a maximum_length any field too long to fit in an XLSX
    cell and report error.

Signed-off-by: Philippe Ombredanne [email protected]

We now check that the content is correctly added before calling
XlsxWriter and report and error if the truncated can be truncated.
Several other adaptations of a cell content are done:

 - Create a combined license expression for expressions.
 - Keep only unique values in lists, preserving ordering.
 - Collect the main attribute of well known scan fields
 - Convert mappings to YAML.
 - Truncate the "description" field to the first five lines.
 - Convert anything unknown to string.
 - Normalize line endings
 - Truncate to a maximum_length any field too long to fit in an XLSX
   cell and report error.

Signed-off-by: Philippe Ombredanne <[email protected]>
@tdruez tdruez merged commit 2a9fda7 into main Jun 22, 2021
@tdruez tdruez deleted the 206-xlsx branch June 22, 2021 17:42
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