Skip to content

Redo node 16 support#45

Closed
0xf0xx0 wants to merge 1 commit intodiscord:masterfrom
0xf0xx0:GingkathFox-patch-1
Closed

Redo node 16 support#45
0xf0xx0 wants to merge 1 commit intodiscord:masterfrom
0xf0xx0:GingkathFox-patch-1

Conversation

@0xf0xx0
Copy link
Copy Markdown

@0xf0xx0 0xf0xx0 commented Aug 15, 2021

Didn’t move to a new branch in #40, fixed here without extraneous changes

@0xf0xx0
Copy link
Copy Markdown
Author

0xf0xx0 commented Aug 15, 2021

Gonna update packages too, don’t merge yet (or merge then merge #41, which also updates the packages)

@0xf0xx0
Copy link
Copy Markdown
Author

0xf0xx0 commented Aug 15, 2021

@icdevin @jhgg sorry for the earlier mixup

@promise
Copy link
Copy Markdown

promise commented Aug 15, 2021

It seems pointless to have two PRs fixing the same issue, and if you're going to also update packages then this is pretty much a copy of #41. Merging both PRs will also have an unnecessary conflict as they have the same changes. Resolving the conflict and merging both PRs will only confuse and has the same effect as merging one and closing the other one.

I would suggest closing this and keep #41 but feel free to wait for a maintainer to give their input instead.

@jhgg
Copy link
Copy Markdown
Contributor

jhgg commented Aug 15, 2021

Closing as dupe of #41

@jhgg jhgg closed this Aug 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants