🌱 Update application analysis CI tests #286
Conversation
Updating structure of test-data and adding kantra anlyze tests also for java binary and java with maven settings file. Related to konveyor#233 Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
aufi
changed the title
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
Signed-off-by: Marek Aufart <[email protected]>
|
Including also test for https://github.com/konveyor-ecosystem/coolstore @ main branch, quarkus and futher updates will follow. |
Signed-off-by: Marek Aufart <[email protected]>
There was a problem hiding this comment.
Maybe we should name this directory tackle-testapp-public so that it's easier to find?
There was a problem hiding this comment.
Also, there seems to be something wrong with kantra here, because we should be getting one issue related to the local-storage-00001 rule that we're not getting in these results. @rromannissen FYI
| <server> | ||
| <id>tackle-testapp</id> | ||
| <username>konveyor-read-only-bot</username> | ||
| <password>GITHUB_TOKEN</password> |
There was a problem hiding this comment.
At what point do we substitute this token? Is it necessary or this is the actual token?
There was a problem hiding this comment.
Good point, for source analysis without dependencies, the settings.xml doesn't matter. For full-analysis it need to be replaced, will update.
Updating structure of test-data and adding kantra anlyze tests also for java binary and java with maven settings file.
Related to #233