Skip to content
#

browsersync

Here are 509 public repositories matching this topic...

langpavel
langpavel commented Oct 24, 2017

Too wide (and falsy) segregation of GraphQL Types and Resolvers

The first example of this is new ObjectType in src/data/queries/index.js importing fields.
Fields should be taken in context of parent type, not as standalone information.

This can ensure some users that custom types cannot have own field resolvers which is false.

In fact, much of fields which can have own resolvers

olets
olets commented May 28, 2019

#477 and a couple different #478 bugs stem from the expectation that setting prebuild: [] or postbuild: [] is a way of saying "prebuild: nothing" and "postbuild: nothing". In fact, the empty array creates problems (see the linked issues for examples)

This should at least be documented. Supporting [] as "nothing" would be an easy enhancement

malinushj
malinushj commented Mar 17, 2018

This could be useful for contributors to test any changes (manually for now) inside the repo, not having to use an external environment.
This folder should be in .npmignore.

Automated testing I think is out of the scope of this project since it's more of a wrapper plugin for Browsersync.
Perhaps some unit tests for the css emissions check?

Improve this page

Add a description, image, and links to the browsersync topic page so that developers can more easily learn about it.

Curate this topic

Add this topic to your repo

To associate your repository with the browsersync topic, visit your repo's landing page and select "manage topics."

Learn more

You can’t perform that action at this time.