Adam Ross
Adam Ross
This sample does not have end-to-end test coverage. This coverage should be added. The existing unit test checks we write the expected log. The essential benefit of an end-to-end test...
This sample does not have end-to-end test coverage. This coverage should be added. A direct health check is minimal, but since this sample demonstrates handling message from Pub/Sub it may...
My application provides the schema I am using for validation. The JsonContext ultimately enforces that all schemas be a local file on disk, though otherwise file_get_contents() can retrieve the URL...
In reviewing the documentation and code I cannot see how Unix-domain sockets ([https://www.postgresql.org/docs/9.5/libpq-connect.html#AEN43017](https://www.postgresql.org/docs/9.5/libpq-connect.html#AEN43017)) are supported. #19 implies it may be supported. Is it currently supported? If not, would you consider...
I am not convinced that any users of GDT are using PHPMD reports. Perhaps we should simplify by dropping it?
The "core" operations of GDT are the things that belong in `grunt help`, that are the high-value, task-oriented features of the system. At present, this is somewhat clouded, and we...
Follow-up to #262 I've been thinking we should not run this as part of the build process, as I wouldn't want changes to code made by CI servers. However, we...
grunt package is versatile, but is fundamentally limited to one type of packaging per project. Furthermore, it's configuration is confusing and it is currently limited to copy operations to move...
This combination of tools could be built into our process to expedite resolving style issues. 1. Does the code reformatting require review for correctness of style, or even error-free functionality?...
Once we are on Node 8 as minimum, let's go ahead and mandate npm v5. This will allow us to use package-lock.json, which is nicer than shrinkwrap. 1. Add this...