Reduce max-length of flake8 #3201
Conversation
There was a problem hiding this comment.
Hi @marameref, Nice job on this PR! It looks like the changes here are all good, but there are a few merge conflicts. Please take a look at this document to understand how to resolve these merge conflicts -> https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-on-github
I appreciate your effort on this PR, so please let me know if you need help resolving the merge conflicts.
|
hi, @yadudoc i believe i have been able to resolve the merge conflicts, the documentation was pretty helpful, thanks you for your aid, pls review and let me know my attention is still needed, if not ... pls merge |
There was a problem hiding this comment.
Looks like all the pending issues here have been resolved. Once the CI tests pass, this PR is good to merge to master. Once again, nice job @marameref!
|
Thank you for the response @yadudoc , am happy to hear that, i am also happy to inform you that all the CI tests passed, I look forward to you merging my PR. Thanks once again |
Description
Please include a summary of the change and (optionally) which issue is fixed. Please also include
relevant motivation and context.
Changed Behaviour
Which existing user workflows or functionality will behave differently after this PR?
none
Fixes
Fixes # (3105)
Type of change
Choose which options apply, and delete the ones which do not apply.
@benclifford I have revamped and corrected the work as requested, pls review