-
Notifications
You must be signed in to change notification settings - Fork 16
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
Package name rewrite #33
Comments
That sounds good to me! I'd prefer not a monorepo since I feel like it's harder to manage, but I don't have a super strong opinion if you think it makes more sense |
I don't have a strong opinion on monorepo, but it is not too bad with pnpm or yarn 2. We can keep them separate for now. I don't have a lot of permissions on the org so I will need you to make the changes. I would also appreciate if you could add an organization secret for the nom publish so I can start migrating the CI. |
Just increased your access level for the github org, so you should be able to do anything you need to set up repos |
Could you send me an email at [email protected]? It's probably easier to coordinate the npm stuff that way |
Ok I am back on this project.
I propose that we change the name of package to
@graphql-query-rewriter/core
.That will allow us to create other packages like
@graphq-query-rewriter/express
and@graphq-query-rewriter/apollo
.We might want to consolidate that in a monorepo or keep them in separate repo.
What do you think @chanind @mattetre
The text was updated successfully, but these errors were encountered: