| opendevreview | Thomas Goirand proposed openstack/mistral master: SSHAction: Add return_result_on_error=True/False https://review.opendev.org/c/openstack/mistral/+/963814 | 10:41 |
|---|---|---|
| zigo | amorin: This version of the patch does what I intended, though I'm not sure yet if the unit tests will pass. I added release note and a small doc change. | 10:42 |
| amorin | zigo perfect, thanks | 12:22 |
| amorin | will review | 12:22 |
| amorin | tests are KO | 12:22 |
| opendevreview | Thomas Goirand proposed openstack/mistral master: SSHAction: Add return_result_on_error=True/False https://review.opendev.org/c/openstack/mistral/+/963814 | 12:52 |
| zigo | That was expected. :) | 12:52 |
| zigo | Hopefully, that's going to be better now. | 12:52 |
| zigo | FYI, I first tried to ask an LLM to write the patch, and it was *really* crap ! I had to rewrite it all or be really ashamed ... :P | 12:54 |
| opendevreview | Thomas Goirand proposed openstack/mistral master: SSHAction: Add return_result_on_error=True/False https://review.opendev.org/c/openstack/mistral/+/963814 | 13:29 |
| opendevreview | Thomas Goirand proposed openstack/mistral master: SSHAction: Add return_result_on_error=True/False https://review.opendev.org/c/openstack/mistral/+/963814 | 13:46 |
| *** __ministry is now known as Guest29080 | 17:04 | |
| amorin | ahah, using LLM is helpful most of the time, but that's also frustrating :) | 20:16 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!