Add a code fix for adding missing seq before {…}#1333
Add a code fix for adding missing seq before {…}#1333edgarfgp wants to merge 3 commits intoionide:mainfrom
seq before {…}#1333Conversation
|
🤖 This is an automated response from Repo Assist. Hi @edgarfgp! This draft PR has been open since December 2024 — just checking in to see if you're still working on it or would like help completing it. The related issue (#1330) is still open, so the feature would be a welcome addition when ready. Let us know if there's anything blocking progress or if you'd like a hand!
|
|
/repo-assist figure out what is wrong with the CI, fix it, rebase the changes on main, and create new PR with all the changes |
|
Pull request created: #1474
|
|
🤖 This is an automated response from Repo Assist. I've analyzed the CI failures in this PR and created a rebased version that fixes them. Root cause of CI failures:
The Fantomas formatter check was failing on both files. Additionally, the branch was behind I've created a new PR with:
The new PR builds cleanly. Credit for the implementation remains with you, @edgarfgp — this is just a rebase + format fix. You can close this PR in favour of the new one if you'd like!
|
|
Will be merged as #1474 |
fixes #1333 CI) (#1474) * Add a code fix for adding missing seq before {…} * more tests * { start..step..finish } * fix: apply fantomas formatting to AddMissingSeq code fix Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Edgar Gonzalez <egonzalez@totallymoney.com> Co-authored-by: Repo Assist <copilot@github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2 participants
Fixes #1330