Aaron Conole via dev <ovs-dev@openvswitch.org> writes: > Sourcery AI suggests: > > Replace m.group(x) with m[x] for re.Match objects > > Suggested-by: SourceryAI <he...@sourcery.ai> > Signed-off-by: Aaron Conole <acon...@redhat.com> > ---
FYI - the pull request got generated: https://github.com/ovsrobot/ovs/pull/5 NOTE: this will only happen if all the initial tests pass (so apply and github report success). BUT, there are a few side effects: 1. A bad log in the pw-ci broke the ci_mon script parsing, so we didn't get a check notification. I've fixed it, so future posts should work. 2. The bot suggested in https://github.com/ovsrobot/ovs/pull/2 that we use m[x] syntax. However, now both coderabbit and Sourcery are advising against m[x] syntax. Given the bot was 'suggesting' instead of providing any code at that moment, I credited it as a 'Suggested-by' but the project itself will need to adopt a better scheme for attributing LLM enhanced patches. 3. The automation to 'close' the PRs isn't there yet. I'll work on it over the next few weeks. I think we can close this particular MR. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
