Clean up testing/contributing#4960
Conversation
|
You can preview documentation at https://esmci.github.io/cime/branch/remove-scripts-regression/html/index.html |
billsacks
left a comment
There was a problem hiding this comment.
Thanks a lot for this cleanup!
|
@billsacks Were you able to get the container to work? I tried on Casper with podman run -it --rm --hostname docker -e CIME_MODEL=cesm -v $PWD:/root/model -v ./storage:/root/storage -w /root/model/cime ghcr.io/esmci/cime:latest bash |
No, I haven't tried it yet. |
- Add missing imports: atexit, functools, logging, get_current_commit, get_model - Fix CIME.utils references to use already-imported utils module - Add TEST_RESULT module-level variable to prevent NameError in cleanup()
|
@samsrabin Go ahead and give podman another try. |
|
My command from above, Based on the README, Do I actually need to build the container first? Based on the README, |
You need to create the
I'll retest this, could come down to how podman is configured on the host systems.
You only need to build if you use |
|
Here's what |
|
Updates:
and then it ends, returning me to the prompt. |
Description
contribute-guide.rst.CONTRIBUTING.md.scripts_regression_tests.py, replaced bypytest.code_checker.py, replaced bypylintviapre-commit.Checklist