Skip to content

Conversation

wajnberg
Copy link

@wajnberg wajnberg commented Sep 5, 2016

Signed-off-by: Moshe Wajnberg [email protected]

@wajnberg
Copy link
Author

wajnberg commented Sep 5, 2016

Hello @knolleary

Could you please review this very simple change ?

Thank you

@coveralls
Copy link

coveralls commented Sep 5, 2016

Coverage Status

Coverage remained the same at 76.566% when pulling a61fda4 on wajnberg:master into f70cd1d on node-red:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.566% when pulling a61fda4 on wajnberg:master into f70cd1d on node-red:master.

@dceejay
Copy link
Member

dceejay commented Sep 5, 2016

We need to be extra careful with non-core nodes, as they could be installed on versions of Node-RED without the required patches... Will this break them ?

Signed-off-by: Moshe Wajnberg <[email protected]>
@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.566% when pulling 9992898 on wajnberg:master into f70cd1d on node-red:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.566% when pulling 9992898 on wajnberg:master into f70cd1d on node-red:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.566% when pulling 9992898 on wajnberg:master into f70cd1d on node-red:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.566% when pulling 9992898 on wajnberg:master into f70cd1d on node-red:master.

@coveralls
Copy link

coveralls commented Sep 18, 2016

Coverage Status

Coverage decreased (-0.03%) to 76.537% when pulling f502749 on wajnberg:master into f70cd1d on node-red:master.

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

Successfully merging this pull request may close these issues.

3 participants