Skip to content

Add debugging tool to tests#18288

Merged
rmuir merged 1 commit intoelastic:masterfrom
rmuir:painless_debug
May 12, 2016
Merged

Add debugging tool to tests#18288
rmuir merged 1 commit intoelastic:masterfrom
rmuir:painless_debug

Conversation

@rmuir
Copy link
Copy Markdown
Contributor

@rmuir rmuir commented May 12, 2016

Today this is done with commented out code... and its painful.

Instead we can add a simple Debugger.toString() for debugging.

@rmuir rmuir added >test Issues or PRs that are addressing/adding tests :Plugin Lang Painless labels May 12, 2016
@jdconrad
Copy link
Copy Markdown
Contributor

Looks great. Thanks!

@rmuir rmuir merged commit 113e239 into elastic:master May 12, 2016
@jdconrad jdconrad mentioned this pull request May 12, 2016
18 tasks
@clintongormley clintongormley changed the title painless: add debugging tool to tests Add debugging tool to tests May 17, 2016
@clintongormley clintongormley added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache and removed :Plugin Lang Painless labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test Issues or PRs that are addressing/adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants