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

Needed Code Refactoring #70

Closed
andersy005 opened this issue May 14, 2019 · 2 comments
Closed

Needed Code Refactoring #70

andersy005 opened this issue May 14, 2019 · 2 comments
Projects

Comments

@andersy005
Copy link
Member

It appears that for the last few months, we've accumulated some technical debt:

  • Lack of necessary user-facing supporting documentation on how to extend intake-esm
  • Delayed refactoring: more code was added without refactoring
  • A few components have become tightly-coupled which resulted in situations where some functions are not modular, making the package not flexible enough to extend without code duplication

As more people are finding intake-esm useful for their own use cases, it would be nice to do some code refactoring soon in order to make intake-esm flexible enough to extend and use.

@andersy005 andersy005 added this to To do in Backlog via automation May 14, 2019
@matt-long
Copy link
Contributor

Clean up test directory to clearly distinguish input from out, file types, etc.

@andersy005
Copy link
Member Author

Addressed in #77

Backlog automation moved this from To do to Done Jun 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Backlog
  
Done
Development

No branches or pull requests

2 participants