Aman-Mittal commented on PR #5927: URL: https://github.com/apache/fineract/pull/5927#issuecomment-4621614770
> > @adamsaghy I think we can also allow upload logs in case of release ready branches. What do you think on that? While on PR we can only upload failure logs but release ready branches i think it would be great practice to keep logs on that. > > I am not sure, i am following you on this one. My reasoning behind this is that. from what i know that will be used by many forks and organizations so it will be beneficial for us in that case if something gone wrong in code despite passing checks and e2e. Sometimes keeping this logs comes handy if there is any logical bugs or any undetected regressions later on. Or for rollback anything lateron. I am not blocking this PR but for observability we need to rethink that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
