Skip to content

Avoid unecessary operations in kb-importer#537

Draft
pedrogalvao wants to merge 85 commits intoeclipse-steady:masterfrom
pedrogalvao:master
Draft

Avoid unecessary operations in kb-importer#537
pedrogalvao wants to merge 85 commits intoeclipse-steady:masterfrom
pedrogalvao:master

Conversation

@pedrogalvao
Copy link
Copy Markdown

Avoid uploading the affected libraries and construct changes when they are already present in the backend.
Delete affected libraries and constructs changes before uploading if the overwrite option is used,

TODOs

  • Tests
  • Documentation

Copy link
Copy Markdown
Contributor

@henrikplate henrikplate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pedrogalvao You accidentally committed java-files.txt and non-compliant-files.txt, please remove them.

@henrikplate henrikplate added the wip Work in progress - Don't merge label Apr 6, 2022
Comment thread kb-importer/src/main/java/org/eclipse/steady/kb/task/ImportVulnerability.java Outdated
Copy link
Copy Markdown
Contributor

@henrikplate henrikplate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comment on ImportVulnerability.java

Comment thread kb-importer/src/main/java/org/eclipse/steady/kb/task/ImportVulnerability.java Outdated
Comment thread kb-importer/pom.xml Outdated
@henrikplate henrikplate marked this pull request as draft September 19, 2022 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wip Work in progress - Don't merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants