Skip to content
This repository has been archived by the owner on Mar 7, 2018. It is now read-only.

Introduce clients package #21

Merged
merged 1 commit into from Jun 29, 2017
Merged

Introduce clients package #21

merged 1 commit into from Jun 29, 2017

Conversation

c-w
Copy link
Contributor

@c-w c-w commented Jun 29, 2017

This pull request improves the organization of our package structure by introducing a wrapper package for all clients. After merging, our folder structure will be much simpler: schemas and resolvers for GraphQL plus clients for third-party integrations.

While looking at all of our require statements, I also fixed a few inconsistencies like always using const and never relying on an implicit Promise object.

@c-w c-w requested a review from jcjimenez June 29, 2017 18:01
@c-w c-w requested a review from Smarker June 29, 2017 20:21
Copy link
Contributor

@Smarker Smarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@c-w c-w merged commit 384ca14 into master Jun 29, 2017
@c-w c-w deleted the clients-refactor branch July 7, 2017 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants