Skip to content

allow method chaining for OdfTable and OdfTableCell #397

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

Merged
merged 1 commit into from
Jun 1, 2025

Conversation

xzel23
Copy link
Collaborator

@xzel23 xzel23 commented Jun 1, 2025

allows method chaining for OdfTable and OdfTableCell, fixes #395

svanteschubert
svanteschubert previously approved these changes Jun 1, 2025
Copy link
Contributor

@svanteschubert svanteschubert left a comment

Choose a reason for hiding this comment

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

Thank you, Axel, for providing the patch!

svanteschubert
svanteschubert previously approved these changes Jun 1, 2025
Copy link
Contributor

@svanteschubert svanteschubert left a comment

Choose a reason for hiding this comment

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

Required a merge of time/date changes in a single line of OdfTableCell

@xzel23
Copy link
Collaborator Author

xzel23 commented Jun 1, 2025

Ah, yes, my master branch has diverged a bit because I can only test the sonar integration on master (a restriction of the free version). I clean this up.

@xzel23
Copy link
Collaborator Author

xzel23 commented Jun 1, 2025

OK, PR is ready to be merged.

@xzel23 xzel23 added the review wanted Four eyes notice more than two label Jun 1, 2025
Copy link
Contributor

@svanteschubert svanteschubert left a comment

Choose a reason for hiding this comment

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

Thanks again, Axel! :-)

@svanteschubert svanteschubert merged commit 96948c6 into tdf:master Jun 1, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review wanted Four eyes notice more than two
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow methods to be chained
2 participants