-
-
Notifications
You must be signed in to change notification settings - Fork 43
chore: remove routify #252
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
Conversation
🦋 Changeset detectedLatest commit: 59666a9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
we can remove the |
Do you mean completely removing the |
But that's actually not true, |
We can just have the In the case of |
Should this still be an object while we are going to be refactoring that? Can we think of any other Should we rename |
Maybe we should just get rid of the field entirely and add a |
Yeah, I was talking about the
|
yeah, it doesn't have to block this PR I'd thought about the setup hook too. The trade-off there is that a hook is more flexible, but can't be displayed easily via a website directory |
But
|
As we decided to ditch #32 for an upcoming more flexible PR, we should remove routify separately.
Reasoning: Routify is nearly unused on npm, and we do recommend creating full kit applications