Permalink
Commits on Feb 17, 2017
  1. Merge pull request #52 from wincent/glh/upgrades

    Upgrade some dependencies
    wincent committed on GitHub Feb 17, 2017
  2. Merge pull request #51 from wincent/glh/interfaces

    Hoist some more types up into "types" package
    wincent committed on GitHub Feb 17, 2017
  3. Bump "types" package to 0.0.7

    wincent committed Feb 17, 2017
  4. Hoist some more types up into "types" package

    Want to break circular dependency of "interface" on "server". ("server"
    depends on "interface", but "interface" depends on a type in "server".)
    
    Specifically, "interface" depends on GraphQLCache, which depends on
    GraphQLRC, which depends on GraphQLConfig, so all three get hoisted up.
    wincent committed Feb 17, 2017
  5. Merge pull request #50 from wincent/glh/depend-utils

    Depend on published graphql-language-service-utils package
    wincent committed on GitHub Feb 17, 2017
  6. Merge pull request #49 from wincent/glh/dead-packages

    Remove unused chai-json-equal and chai-subset packages
    wincent committed on GitHub Feb 17, 2017
  7. Merge pull request #48 from wincent/glh/extract-utils

    Extract utils package
    wincent committed on GitHub Feb 17, 2017
  8. Remove unused chai-json-equal and chai-subset packages

    These were likely only in here because of copy-pasta, so out they go.
    wincent committed Feb 17, 2017
  9. Fix Flow error

    This one was a legitimate mis-typing, so I am not sure why Flow used to
    be blind to the error when this exact same file lived two levels higher
    in the repo.
    
    Oh well. ¯\_(ツ)_/¯
    
    Confirmed that the sole caller of `validateWithCustomRules` is indeed
    passing in a `DocumentNode` and not `ASTNode`, because the value is
    coming from `parse` in the `graphql` package (which returns a
    `DocumentNode`).
    wincent committed Feb 17, 2017
  10. Remove @flow directives from tests

    We will likely want to typecheck these later, but for now I just want to
    quickly reorganize the files that we have, so I'm going to strip the
    annotation.
    
    Note that I could also have modified the .flowconfig, but I prefer not
    to do a blanket exclusion of all __tests__, so that we can leave open a
    doorway for incremental adoption of Flow on a file-by-file basis in the
    future.
    wincent committed Feb 17, 2017
  11. Create index for utils package

    Note that I'm dumping `SchemaReference`, which is unused, and came over
    from codemirror-graphql. As that's the only caller/user of the
    functionality, the functionality may as well live over there.
    wincent committed Feb 17, 2017
  12. Fix lints in mocha-bootload

    wincent committed Feb 17, 2017
  13. Get tests passing by breaking dependency on other packages

    Don't depend on the server package (which in turn depends on the config
    package). This allows us to avoid an otherwise very hard to disentangle
    circular dependency.
    
    On the down side, this means the test is no longer the end-to-end
    integration test that it was before, in some sense, but the argument
    could be made that it never should have been (it's a unit test). Also,
    note that it wasn't running before because of the "-spec.js" extension
    (we only run files matching "-test.js") so its value was limited anyway.
    
    Let's replace this with an actual integration test in the top-level
    package later on.
    wincent committed Feb 17, 2017
  14. Tweak Flow build script to only deal with .js files

    This stops it from trying to copy "__tests__" needlessly.
    
    Note I am fixing a copied script here, but I have the option of making
    the duplication go away for the most part (only the "types" package will
    need its own copy, because everything depends on that).
    wincent committed Feb 17, 2017
  15. Move utils files down into packages directory

    This is enough to get a working "package". Still to do:
    
    - Adjust `"main"` and set up convenient exports for the package.
    - Confirm that `__tests__` files are excluded from `dist/`, which means
      our existing non-recursive build step will work just fine.
    - Get the tests running (will require some deps to be added and a mock
      for the server cache, which we cannot add to this project without
      creating a circular dependency).
    wincent committed Feb 17, 2017
  16. Merge pull request #47 from wincent/glh/depend-config

    Depend on published graphql-language-service-config package
    wincent committed on GitHub Feb 17, 2017
  17. Merge pull request #46 from wincent/glh/extract-config

    Extract graphql-language-service-config package
    wincent committed on GitHub Feb 17, 2017
  18. Extract graphql-language-service-config package

    Like we did for graphql-language-service-parser, and the same
    disclaimers about duplication of build/util functionality apply (I'll
    address that shortly).
    wincent committed Feb 17, 2017
  19. Merge pull request #45 from wincent/glh/fix-lint

    Fix a long-line lint
    wincent committed on GitHub Feb 17, 2017
  20. Merge pull request #44 from wincent/glh/depend-parser

    Depend on published graphql-language-service-parser module
    wincent committed on GitHub Feb 17, 2017
  21. Fix a long-line lint

    wincent committed Feb 17, 2017
  22. Merge pull request #43 from wincent/glh/extract-parser

    Extract graphql-language-service-parser
    wincent committed on GitHub Feb 17, 2017
  23. Fix up re-exports of defaults

    Verified with a `node` console that they all work.
    wincent committed Feb 17, 2017
  24. Extract graphql-language-service-parser

    Some duplication with graphql-language-service-types, which I will
    address in a follow-up.
    wincent committed Feb 17, 2017
Commits on Feb 16, 2017
  1. Merge pull request #42 from graphql/wincent-patch-1

    Fix awkward phrasing in README
    wincent committed on GitHub Feb 16, 2017
  2. Fix awkward phrasing in README

    "The Watchman" sounds like some kind of comic book villain.
    wincent committed on GitHub Feb 16, 2017
  3. Merge pull request #41 from wincent/glh/types-dep

    Depend on graphql-language-service-types
    wincent committed on GitHub Feb 16, 2017
  4. Depend on graphql-language-service-types

    Depend on the extracted package, instead of the local copies of the
    files. To develop against the local copy you have to use `npm link` or
    `yarn link` for now, but I'll try out Lerna in a bit which will smooth
    over some of the rough edges with the multi-package development process.
    wincent committed Feb 16, 2017