-
Notifications
You must be signed in to change notification settings - Fork 289
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
Sandraros/check demo regressions #902
Conversation
1) change many demo programs to make lcl_excel_generator inherit from zcl_demo_excel_generator instead of implementing zif_demo_excel_generator. 2) replace sy-datum with zcl_demo_excel_generator=>get_date_now( ), and sy-uzeit with zcl_demo_excel_generator=>get_time_now( ), so that comparison can operate on same fixed data
Co-authored-by: Abo <andrea@borgia.bo.it>
I'll do the full review over the coming days, stay tuned. |
Review time... |
@AndreaBorgia-Abo I forgot to say that I was working on a second simplest version to check demo regressions, whose advantage is to not change the demo programs. It's simply based on the Excel files created on the frontend by program |
Sheesh, ended up using this PR as a guinea pig for a possible abapGit bug ... I'll check the newer version first, then! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should probably be closed in favour of the newer #940
Closing in favor of other PR #940 |
Fix #894