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

refactor: v7 #2332

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

refactor: v7 #2332

wants to merge 22 commits into from

Conversation

kyranet
Copy link
Member

@kyranet kyranet commented Jul 27, 2022

This is a heavy WIP PoC of Skyra v7.

On a side note, migrations are going to be fun to do.

@kyranet kyranet added this to the Skyra V7 milestone Aug 13, 2022
@HigherOrderLogic
Copy link

Hope you guy don't get mad for necro-ing this, but is this still WIP? 😅

@kyranet
Copy link
Member Author

kyranet commented Jul 20, 2024

Well, this is a promising version of Skyra that predates the actual v7 plans, we went for the alternative route of doing the changes incrementally, and make Skyra gradually lighter so she's easier to rewrite.

Current milestone is v6.5, which is "halfway" v6 (keeps discord.js) and v7 (new architecture). I plan to eventually rewrite Skyra into a hybrid of @skyra/http-framework with @discordjs/core, but there are many challenges around it, which I tried to solve with this PR, but I think this approach is not completely right.

Nevertheless, I want to investigate some things before trying this first, but for now, the migration to Prisma and minimising Skyra's memory are priority atm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants