Skip to content

Conversation

@BethanyG
Copy link
Member

@BethanyG BethanyG commented Oct 29, 2021

The new PR-Commenter for the repo, codename: Porcupine Revenger. Made together with @J08K ❤️

Splitting this into two parts, for sharing of REP purposes.

MERGE FIRST PLEASE

@BethanyG BethanyG requested a review from J08K October 29, 2021 23:27
@BethanyG BethanyG added the x:size/large Large amount of work label Oct 29, 2021
@BethanyG BethanyG changed the title [Automation]: Added snippets to the auto-pr-commenter. 2/2 [New Workflow]: Added snippets to the auto-pr-commenter. 2/2 Oct 29, 2021
@BethanyG BethanyG requested a review from a team October 29, 2021 23:36
Copy link
Member

@kotp kotp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick read through. "whitespace issues" noticed.

It looks like it has been carefully proofread!

@BethanyG
Copy link
Member Author

Ok. I think I got all those edits! LMK if I didn't. This was a weird one. I had to resort to using VIM to edit the file -- PyCharm would not save the edits, and git would not pick them up. It was very strange.

@kotp
Copy link
Member

kotp commented Oct 30, 2021 via email

@BethanyG
Copy link
Member Author

@kotp -- does that mean this is ready -- or are there still things to be addressed? Let Me Know. Thanks!

@BethanyG BethanyG requested a review from kotp October 30, 2021 16:31
@BethanyG
Copy link
Member Author

@kotp -- if there is nothing more to address, please approve so I can merge this and the follow-on PR. Many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:size/large Large amount of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants