-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added design documentation to week 13 deliverabels #238
Conversation
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.
In 4.2.2 there are typos it is UDP instad of UPD. You can also parse the text in ChatGPT it will fix them and make it a bit more readable (I also do this with my texts ;))
The UDP does not communicate with the executor. The UDP-server is mainly there to check if the parameters were correctly overriden more specific just the targetClasses. The executor is a process started by intellij in which the command line that we built with the user input is executed. When the process is done the ExecutionDoneProcessListener.processTerminated function is called by intellij and this triggers actions like update the reports, archive the run,... (@lheimbs please correct me if I am wrong)
If you have time you can also add some stuff about the new task PitmuationmateStatusCheckTask in the override plugin. This task is executed in our intellij plugin before each pitest run to check if the necessary plugins are present.
Adressed changes requested by @nikomall34. Unfortunatly i dont have time before our meeting to add the mentioned PitmuationmateStatusCheckTask into the diagram and the description |
There is still the UPD typo in the diagram as far as i can see |
Thanks for checking, i uploaded the wrong version there |
Added pdf version of wiki page for the design documentation. Also changed the Design and Tech Stack pages to reflect the changes.