-
Notifications
You must be signed in to change notification settings - Fork 62
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
Improve the Pass system #1645
Improve the Pass system #1645
Conversation
…d support "soft" ExecuteBefore depndencies.
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/TranslationConfiguration.kt
Show resolved
Hide resolved
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/TranslationConfiguration.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only minor formatting issues.
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/MermaidHelper.kt
Outdated
Show resolved
Hide resolved
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/MermaidHelper.kt
Outdated
Show resolved
Hide resolved
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/MermaidHelper.kt
Outdated
Show resolved
Hide resolved
cpg-core/src/main/kotlin/de/fraunhofer/aisec/cpg/MermaidHelper.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except the minor formatting issues int he mermaid file that should be resolved first.
Quality Gate passedIssues Measures |
This PR cleans up the pass ordering system. It removes some logic bugs and tries to keep a strict separation between files (TranslationConfiguration does not do the ordering logic, ...)
New features:
ExecuteLate
annotation.ExecuteBefore
now hassoft
/hard
dependencies.Pass
es (theirKClass
) can now be queried forisFirstPass
,softDependencies
, ...Fixes #1641
Todo;
workingList
soft
/hard
forExecuteBefore
as requested by @KuechA