Skip to content

Clean up testing/contributing#4960

Open
jasonb5 wants to merge 4 commits intomasterfrom
remove-scripts-regression
Open

Clean up testing/contributing#4960
jasonb5 wants to merge 4 commits intomasterfrom
remove-scripts-regression

Conversation

@jasonb5
Copy link
Copy Markdown
Collaborator

@jasonb5 jasonb5 commented Apr 9, 2026

Description

  • Cleans up contribute-guide.rst.
  • Cleans up CONTRIBUTING.md.
  • Removes scripts_regression_tests.py, replaced by pytest.
  • Remove old code_checker.py, replaced by pylint via pre-commit.

Checklist

  • My code follows the style guidelines of this project (black formatting)
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added tests that exercise my feature/fix and existing tests continue to pass
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding additions and changes to the documentation

@jasonb5 jasonb5 requested review from billsacks and jgfouca April 9, 2026 20:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

@jasonb5 jasonb5 requested a review from samsrabin April 9, 2026 20:58
Copy link
Copy Markdown
Member

@billsacks billsacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this cleanup!

@samsrabin
Copy link
Copy Markdown
Contributor

@billsacks Were you able to get the container to work? I tried on Casper with podman, and it just stopped after a bunch of "Copying blob ... Done" messages.

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

@billsacks
Copy link
Copy Markdown
Member

Were you able to get the container to work?

No, I haven't tried it yet.

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.

Testing instructions in CONTRIBUTING.md are incorrect/incomplete

4 participants