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

Update cfenv version #459

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kenime
Copy link
Contributor

@kenime kenime commented Jul 10, 2019

Update cfenv version to fix npm audit report

Update cfenv version to fix npm audit report
@chughts
Copy link
Member

chughts commented Jul 10, 2019

Have you run any regression tests to verify the code continues to work with the update in the dependancy?

@kenime
Copy link
Contributor Author

kenime commented Jul 11, 2019

I have performed regression on my current project (which only uses assistant). From cfenv's change log there is no breaking change, so I assumed it will work fine for other nodes as well.

Any standard way to perform regression test? I can execute it after project hours...

@chughts
Copy link
Member

chughts commented Jul 11, 2019

Nothing automated. Just a couple of Node-RED instances with test flows, one with connected services, and one without.

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

Successfully merging this pull request may close these issues.

2 participants