openpipelines-bio / openpipeline

https://openpipelines.bio
MIT License
27 stars 13 forks source link

Move common code to a separate script #413

Open VladimirShitov opened 1 year ago

VladimirShitov commented 1 year ago

Currently, there is a code, which is repeated across different components. The examples that I faced are:

Code copy-pasting is not a good practice. If we decide to change a logging format or a bug is found (see recent PR https://github.com/openpipelines-bio/openpipeline/pull/385), we must change the code in each component. To prevent this, we should move common code to a separate script or component.

I created such a script in src/ directory in this PR: https://github.com/openpipelines-bio/openpipeline/pull/385. After merging it, I can move other reused code to this script. Other suggestions about common code location or examples of copy-pasting are welcome :)

VladimirShitov commented 1 year ago

Upd: the folder for the utils scripts is now src/utils