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

Fixes config ignoring paths containing tests #4621

Merged
merged 1 commit into from
May 6, 2024
Merged

Fixes config ignoring paths containing tests #4621

merged 1 commit into from
May 6, 2024

Conversation

jasonb5
Copy link
Collaborator

@jasonb5 jasonb5 commented May 3, 2024

When CIME's customize config would load it would ignore paths
containing tests. This PR fixes the ignore pattern and only applies
it under the cime_config/customize directory.

Test suite: pytest CIME/tests/test_unit*
Test baseline: n/a
Test namelist changes: n/a
Test status: n/a

Fixes #4611
User interface changes?: N
Update gh-pages html (Y/N)?: N

@xylar
Copy link

xylar commented May 5, 2024

Thanks @jasonb5, I really appreciate this fix a lot!

@jgfouca jgfouca merged commit 84782b7 into master May 6, 2024
7 checks passed
@jgfouca jgfouca deleted the fix-config branch May 6, 2024 15:26
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.

You can't work from a directory that has tests in the name, it confuses CIME.
3 participants