Skip to content

feat: allow executing JS funcitons on cleanup #4581

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

Merged
merged 1 commit into from
May 7, 2019

Conversation

rosen-vladimirov
Copy link
Contributor

In some cases it is required to execute some JS functions to clean used resources by CLI. Add an easy way to specify JS files which should be executed by the cleanup process.
Each JS action is defined by a JS File, which should be required, data that will be passed to the default exported function in the file and timeout - if the action cannot be executed for specified time (3 seconds is the default), the child process in which the JS action is executed will be killed.

PR Checklist

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

In some cases it is required to execute some JS functions to clean used resources by CLI. Add an easy way to specify JS files which should be executed by the cleanup process.
Each JS action is defined by a JS File, which should be required, data that will be passed to the default exported function in the file and timeout - if the action cannot be executed for specified time (3 seconds is the default), the child process in which the JS action is executed will be killed.
@rosen-vladimirov rosen-vladimirov added the no-changelog The issue is present only in `next` versions and was not included in official version. label May 3, 2019
@rosen-vladimirov rosen-vladimirov added this to the 5.4.0 milestone May 3, 2019
@rosen-vladimirov rosen-vladimirov self-assigned this May 3, 2019
@cla-bot cla-bot bot added the cla: yes label May 3, 2019
@ghost ghost added the new PR label May 3, 2019
@rosen-vladimirov
Copy link
Contributor Author

test cli-smoke

@rosen-vladimirov rosen-vladimirov merged commit 66ce514 into master May 7, 2019
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/improve-cleanup branch May 7, 2019 05:42
@ghost ghost removed the new PR label May 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes no-changelog The issue is present only in `next` versions and was not included in official version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants