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

Better Debug Logging #65

Closed
nmccready opened this issue Mar 13, 2018 · 3 comments
Closed

Better Debug Logging #65

nmccready opened this issue Mar 13, 2018 · 3 comments

Comments

@nmccready
Copy link
Contributor

Optional debug logging would go a long way in figuring out what is going on in the clasp API when things do not work as expected. A very common practice in the node community is to utilize debug. If you want to gain lazy logging please have a look at debug-fabulous (shameless plug).

Expected Behavior

export DEBUG="@google/clasp*"

@google/clasp:getProjectSettings oh hi I am a debug statement

To turn on useful optional log messages.

Actual Behavior

Only errors are logged.

Examples

https://github.com/gulp-sourcemaps/gulp-sourcemaps/blob/master/src/debug.js#L1

https://github.com/gulp-sourcemaps/gulp-sourcemaps/blob/master/src/init/index.js#L49

@nmccready nmccready changed the title Better Debug logging feature request: Better Debug logging Mar 13, 2018
@grant grant changed the title feature request: Better Debug logging Better Debug Logging Mar 22, 2018
@grant
Copy link
Contributor

grant commented Mar 30, 2018

PRs or detailed designs are welcome. Debugging clasp is an issue for me as well, and would like to see improvement.

@nmccready
Copy link
Contributor Author

I'll try to get to a PR when I get back to working on google app scripts again. Thanks.

@grant grant added this to the Clasp V2 milestone Apr 6, 2018
@grant
Copy link
Contributor

grant commented Jun 1, 2018

Clasp has been split into multiple files with a testing framework that tests individual utility methods.

I don't know exactly what debug logging this issue would like to see, but I don't think contributors would need a DEBUG setting to make debugging clasp easier. Better testing is preferred over better logging.

I'll close this issue, but feel free to create a new one when there's something specific that should be added to clasp.

@grant grant closed this as completed Jun 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants