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

[R/print] update print returns #258

Merged
merged 4 commits into from
Jul 5, 2022
Merged

[R/print] update print returns #258

merged 4 commits into from
Jul 5, 2022

Conversation

JanMarvin
Copy link
Owner

Initial update to print returns

As discussed previously: in the future we might want to add something similar to the sheet waiver for output messages. Something like this:

  • create Workbook. Output is a workbook
  • add a worksheet foo. Output is a workbook \n+ worksheet foo
  • add data bar to worksheet foo. Output is a workbook \n+ worksheet foo \n+ data bar
  • etc

There is absolutely no need to rebuild every C++ file all the time. Imho this option is really mad nonsense. It simply creates a waste of time and energy.
Previously no names were printed. Instead media and images were returned. Now only sheet names and sheet write order are returned.
@JanMarvin JanMarvin merged commit 7a5cab4 into main Jul 5, 2022
@JanMarvin JanMarvin deleted the print_pt1 branch July 5, 2022 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant