Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[New Workflow]: PR-Commenter Part 1/2 #2706

Closed
wants to merge 65 commits into from

Conversation

J08K
Copy link
Member

@J08K J08K commented Oct 29, 2021

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

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

DO NOT MERGE until PR #2707 has been merged.

J08K and others added 30 commits September 20, 2021 15:05
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
J08K and others added 13 commits October 14, 2021 09:12
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
@J08K J08K added do not merge 🚧 Don't merge until this label is removed. x:size/large Large amount of work labels Oct 29, 2021
@BethanyG BethanyG self-requested a review October 29, 2021 23:36
Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

🌟 🌈 Thank you for all your work on this. Much appriciated!

@BethanyG
Copy link
Member

Uh - oh. @J08K -- any chance you can do some fetch + rebase magic to resolve the conflicts? Let me know...or I can try from my side....

@BethanyG
Copy link
Member

@J08K -- Apologies. I cannot effectively rebase this. Could you see if you can resolve the merge conflicts in your fork?

@J08K J08K requested a review from BethanyG October 30, 2021 20:23
@J08K
Copy link
Member Author

J08K commented Oct 30, 2021

Closing this issue, because some weird stuff is happening with merge conflicts... Note to self: Be scared when you see so many commits that shouldn't be there.

@J08K J08K closed this Oct 30, 2021
@J08K J08K deleted the J08K-New-Automation branch October 30, 2021 20:51
@BethanyG BethanyG removed the do not merge 🚧 Don't merge until this label is removed. label Jul 14, 2023
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.

2 participants