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

debug output that puzzles me #20

Open
osher opened this issue Nov 9, 2016 · 2 comments
Open

debug output that puzzles me #20

osher opened this issue Nov 9, 2016 · 2 comments

Comments

@osher
Copy link

osher commented Nov 9, 2016

https://github.com/apigee-127/bagpipes/blob/master/lib/bagpipes.js#L219

perhaps you mean
debugContent('output (context[%s]): %j', target, context[target]); ?

@theganyo
Copy link
Contributor

theganyo commented Nov 9, 2016

I don't think so? What are you seeing?

@osher
Copy link
Author

osher commented Nov 10, 2016

 pipes:content output (context[result]): undefined +3ms

sure it's undefined. the output is in another attribute (result), named in fittingDef.output (seen here + the if sttmt that comes after)

I think that in some cases I even got to see the output of the previous pipe... 😛

The names here are rather confusing. I'd rather fittingDef.output to be fittingDef.outputTo, fittingDef.outputsTo, fittingDef.saveAs, fittingDef.savesAs, fittingDef.target, or even fittingDef.memo, to be compatible with that other fitting, but this output thing is all over the place and ambiguous by nature... 🤕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants