Skip to content

Whitespaces indentation header#62

Open
eanokian wants to merge 7 commits intopharmaverse:mainfrom
eanokian:whitespaces_indentation_header
Open

Whitespaces indentation header#62
eanokian wants to merge 7 commits intopharmaverse:mainfrom
eanokian:whitespaces_indentation_header

Conversation

@eanokian
Copy link
Copy Markdown

this PR impacts the RTF exporter and replaces, in header col 1, the whitespaces by the left-indentation RTF raw markup. The conversion ratio is: for every 1 leading whitespace in the header col 1, insert a left-indentation of 90 twips (approx 0.06 inches, so 2 whitespaces correspond to 0.125 inches), and remove any leading whitespaces.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this needs tests as well

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

added unit tests for replace_lead_whitespaces_ind()

@kpagacz
Copy link
Copy Markdown
Collaborator

kpagacz commented Mar 24, 2026

Fix the conflicts, rebase and this should now pass the lints

@kpagacz kpagacz self-requested a review March 25, 2026 08:22
…gen example error, seq_len() to seq_along(),
@eanokian
Copy link
Copy Markdown
Author

I just fixed the CIICD errors, I hope I addressed them all, apologies for that @kpagacz

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