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.

Sign up for GitHub

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

Jump to bottom

Add Pipelines as plugins #91

Closed
tdruez opened this issue Feb 9, 2021 · 0 comments
Closed

Add Pipelines as plugins #91

tdruez opened this issue Feb 9, 2021 · 0 comments
Assignees
@tdruez
Labels
enhancement New feature or request
Milestone
1.1

Comments

@tdruez
Copy link
Contributor

tdruez commented Feb 9, 2021 •
edited by pombredanne
Loading

Until now pipelines have been plain Python scripts living a specific Python module. To improve modularity, pipelines should be installable instead as Python packages that one can add to a scancode.io installation.
A simple design is to use setuptools entry points as way for a Python package to signal it is contributing a pipeline.
@tdruez tdruez added this to the 1.1 milestone Feb 9, 2021
@tdruez tdruez self-assigned this Feb 9, 2021
tdruez added a commit that referenced this issue Feb 9, 2021
@tdruez
Replace pipeline loading using the importlib_metadata entry_points #91
9db6f32 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 9, 2021
@tdruez
Add a register_pipeline method to simplify registration #91
3aee4c7 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 12, 2021
@tdruez
Fix the Pipeline.get_name method and add unit tests #91
8b2a16c 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 12, 2021
@tdruez
Do not automatically start the next pipeline on task success #91
e08ce88 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 12, 2021
@tdruez
Add unit tests for the new Run model methods #91
2a84da6 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
@pombredanne pombredanne changed the title Add Pipelines as plugging Add Pipelines as plugins Feb 13, 2021
@pombredanne pombredanne added the enhancement New feature or request label Feb 13, 2021
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Refine docs #91
268285a 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Rename is_pipeline_subclass to is_pipeline #91
4e936a6 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Use ABC data format for the pipelines details action API #91
2b7d786 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Rename the Run.pipeline to pipeline_name #91
df6dc0d 
Apply that convention everywhere

Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Update the documentation #91
7c29986 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Consolidate naming convention in multiple places #91
5d0726b 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Remove commented line #91
a44296f 
Signed-off-by: Thomas Druez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Feb 15, 2021
@tdruez
Pipeline registration through distribution entry points #91
aa6ba17 
* Replace pipeline loading using the importlib_metadata entry_points #91

Signed-off-by: Thomas Druez <tdruez@nexb.com>

* Add a register_pipeline method to simplify registration #91

Signed-off-by: Thomas Druez <tdruez@nexb.com>
@tdruez tdruez closed this as completed Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tdruez tdruez

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.1
Development

No branches or pull requests
2 participants
@tdruez @pombredanne