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

CF utilities consolidation #774

Open
daltonkell opened this issue Mar 18, 2020 · 2 comments
Open

CF utilities consolidation #774

daltonkell opened this issue Mar 18, 2020 · 2 comments
Assignees

Comments

@daltonkell
Copy link
Contributor

There are two modules proclaiming to be utilities for the CF specification checks:

  • cf/util.py
  • cfutil.py

Both contain functionality to help with CF checks. They should be consolidated into one module. I propose consolidation into cf/util.py.

@benjwadams
Copy link
Contributor

While I agree this was a confusing decision, if I recall correctly a number of plugins may rely on modules being located in certain places. Changing the location of functions may break functionality for those existing checkers as was the case here: ioos/cc-plugin-glider#39

@daltonkell
Copy link
Contributor Author

Perhaps we could scope out what might need to be done for these plugins to align everything

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants