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

Chore/monorepo #200

Merged
merged 7 commits into from
Jun 30, 2020
Merged

Chore/monorepo #200

merged 7 commits into from
Jun 30, 2020

Conversation

rhyslbw
Copy link
Contributor

@rhyslbw rhyslbw commented Jun 30, 2020

This PR refactors the codebase to move code into isolated packages, in preparation for adding additional API segments, and now supports the use-case of importing only the executable schema into an existing server. It's also the first step to releasing the hard dependency on Hasura and PostgreSQL, to open up more lightweight deployments scenarios with other datasources.

@rhyslbw rhyslbw added the enhancement New feature or request label Jun 30, 2020
@rhyslbw rhyslbw self-assigned this Jun 30, 2020
@rhyslbw rhyslbw requested a review from craigem June 30, 2020 05:27
@rhyslbw
Copy link
Contributor Author

rhyslbw commented Jun 30, 2020

@craigem Can you test the nix service operates as expected? AFAIK the only difference here is the location of the build output

Edit: Nevermind, the hydra build is passing 👍

@rhyslbw rhyslbw merged commit 5818ba9 into master Jun 30, 2020
@craigem
Copy link
Contributor

craigem commented Jun 30, 2020

The changes look sound to me. Will sort some testing.

rhyslbw added a commit that referenced this pull request Jun 30, 2020
@rhyslbw rhyslbw deleted the chore/monorepo branch June 30, 2020 08:16
rhyslbw added a commit that referenced this pull request Jul 1, 2020
* Fixes a broken path introduced in #200

* fix required jobset

Co-authored-by: Michael Bishop <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants