Adding reset_kc to setup_kernel

[danielfrg]

Matching what async_setup_kernel does.

[MSeal]

This is a decent idea, thanks for opening a PR. I would like to have a test confirming the new keyword works as expected though.

[danielfrg]

I will add a test soon, I edited this directly on GitHub 🙈

Let me know what you prefer in terms of name/functionality.
My opinion: I agree that cleanup_kc is better.

I can also clean up async_setup_kernel as part of this, just let me know.

If you want to maintain the current convention I guess the default should be cleanup_kc =True, thats what happens now without the flag and seams reasonable to be because it's a contextmanager.

[MSeal]

@davidbrochart what do you think about changing reset_kc to cleanup_kc?

[davidbrochart]

Yes it is more coherent to change it to cleanup_kc. It's relatively new so I think we can afford that change.

[danielfrg]

I cleaned changed reset_kc to cleanup_kc on the *setup_kernel functions and added a test_cleanup_kernel_client test.

Cleaned up a bit reset_kc on execute.

I think this should make it more coherent.

[MSeal]

Thanks!