papercast icon indicating copy to clipboard operation
papercast copied to clipboard

Change plugin imports to from papercast.types.plugin import Class

Open papercast-dev opened this issue 2 years ago • 0 comments

We currently import stuff from plugins with

from papercast.processors import ArxivProcessor, SemanticScholarProcessor

This would change to

from papercast.processors.arxiv import ArxivProcessor from papercast.processors.semanticscholar import SemanticScholarProcessor

Pros:

  • Possibility to simplify static code analysis?
    • Take a look at processors/init.py and see whether this would actually simplify anything. See #9
  • Maybe make it clearer where each object is coming from

Cons:

  • Import statement is not as clean, importing types from several locations take up multiple lines

papercast-dev avatar Apr 20 '23 21:04 papercast-dev