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

clear statusView before script execution #180

Merged
merged 1 commit into from
Mar 16, 2021
Merged

clear statusView before script execution #180

merged 1 commit into from
Mar 16, 2021

Conversation

nhnicwaller
Copy link
Contributor

If the status field still contains info or error from the previous script run, it won't be relevant after running a new script, so the previous status should be immediately evicted.

Fixes #179

If the status field still contains info or error from the previous script run, it won't be relevant after running a new script, so the previous status should be immediately evicted
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@nhnicwaller
Copy link
Contributor Author

@IvanMathy can I get this merged? 🙂

@IvanMathy
Copy link
Owner

Hey there, great catch thank you! Sorry about the delay in getting this through, I appreciate the contribution

@IvanMathy IvanMathy merged commit 918c0cb into IvanMathy:main Mar 16, 2021
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.

Error message is not cleared after re-running script
2 participants