certego / .github

CI repo for Certego DevOps
https://www.certego.net
5 stars 0 forks source link

Print debug informations for detect_changes #83

Closed eugenioseveri closed 1 year ago

eugenioseveri commented 2 years ago

I suggest to print, for each step (backend, frontend) of the detect_changes action, the following data for debugging purposes:

0ssigeno commented 2 years ago

If you need to debug and know which files have been detected as changed, you can rerun the job in debug mode

eugenioseveri commented 2 years ago

I didn't know about additional messages in debug mode. I'll try it, maybe it can cover this use case. Anyway, this is a use case for the number of changed lines: last week there was a new bug in the production CI in the "detect-changes" step which caused the tests not to run. It would have been easier to debug if we had known that changes not only weren't detected, but the variable was not initialized (because it was not exported through reusable workflows). Adding this info just requires a simple "echo" command of that variable.

0ssigeno commented 2 years ago

I still don't see how know the number of changed lines will make any that you described easier. Do you mean the files that detect_changes detected for both backend and frontend?

eugenioseveri commented 2 years ago

In last week's case, we assumed there were no bugs in the workflow (since the previous version worked). Having the number of changed lines would have helped identifying the bug earlier, because that variable was not initialized. But also having the list of changed files could be useful to check what changes the action detects. 😛